secrets webhook doesn't work without command

Describe the bug:
i've a simple deployment
        vault.security.banzaicloud.io/vault-addr: "http://vault:8200"         vault.security.banzaicloud.io/vault-skip-verify: "true"
file (with the above annotations), that will retrieve some values from vault, it works fine and the webhook-container injects the secret fine and turn it's own status to be complete,  if i passed -container-command value
containers:       - name: alpine         image: alpine         command: ["sh", "-c", "echo $AWS_SECRET_ACCESS_KEY && echo going to sleep... && sleep 10000"]         env:         - name: AWS_SECRET_ACCESS_KEY           value: "vault:secret/data/accounts/aws#AWS_SECRET_ACCESS_KEY"
but once i remove this container-command and just pass envs to the container
 containers:       - name: alpine         image: my-company/another-container-that-expects-AWS_KEY-env         env:         - name: AWS_SECRET_ACCESS_KEY           value: "vault:secret/data/accounts/aws#AWS_SECRET_ACCESS_KEY"
the webhook-container doesn't even start or inject any secrets, and i get the value as a string.
Expected behaviour:
i expected vault-webhook to inject the secrets in my container,
Steps to reproduce the bug:
apiVersion: apps/v1 kind: Deployment metadata:   name: hello-secrets spec:   replicas: 1   selector:     matchLabels:       app: hello-secrets   template:     metadata:       labels:         app: hello-secrets       annotations:         vault.security.banzaicloud.io/vault-addr: "http://vault:8200"         vault.security.banzaicloud.io/vault-skip-verify: "true"         # vault.security.banzaicloud.io/vault-tls-secret: "vault-tls"     spec:       serviceAccountName: default       containers:         - name: image           image: image-that-expects-an-env           env:             - name: ENV_STRING               value: "vault:secret/data/HELLO/SECRET#KEY"       imagePullSecrets:         - name: regcred
Environment details:

Kubernetes version: v1.15.3
on prem cluster
vault-secrets-webhook	chart-v: 0.5.5        	app-v:0.5.1
Install method (static yaml files (operator-rbac, operator, rbac, cr)):
Logs from the misbehaving component (and any other relevant logs):
/kind bug

@hacktron95  Could you provide us logs of the vault-secrets-webhook pod?

hello @pbalogh-sa , sorry for missing that
vault-secrets-webhook time="2019-11-26T08:00:01Z" level=info msg="found no credentials for registry index.docker.io, assuming it is public"                                                                                vault-secrets-webhook time="2019-11-26T08:00:01Z" level=info msg="I'm using registry https://index.docker.io"                                                                                vault-secrets-webhook 2019/11/26 08:00:01 registry.ping url=https://index.docker.io/v2/                                                                                vault-secrets-webhook 2019/11/26 08:00:04 registry.manifest.get url=https://index.docker.io/v2/owner/repo/manifests/production repository=owner/repo reference=production                                                                                vault-secrets-webhook time="2019-11-26T08:00:06Z" level=error msg="admission webhook error: cannot download manifest for image: Get https://index.docker.io/v2/owner/repo/manifests/production: http: non-successful response (status=401 body=\"{\\\"errors\\\":[{\\\"code\\ \":\\\"UNAUTHORIZED\\\",\\\"message\\\":\\\"authentication required\\\",\\\"detail\\\":[{\\\"Type\\\":\\\"repository\\\",\\\"Class\\\":\\\"\\\",\\\"Name\\\":\\\"owner/repo\\\",\\\"Action\\\":\\\"pull\\\"}]}]}\\n\")"
this is what it keeps printing, nothing else
despite the fact that the cluster has the docker creds and it already pulled the image and ran it.

@hacktron95 The problem is the webhook hasn't any permission to the registry due to the imagePullsecret regcred doesn't contain proper docker credentials. In this case, the webhook cannot get the command defined in the image from the registry. If you define the command explicitly, the webhook won't try to get the command from the registry.
Is the docker API version and credentials properly defined in your regcred secret?

@pbalogh-sa thanks for your reply, at first i was passing my docker hub creds as a token
then i passed it as a normal secrets kubectl create secret
in both cases, i can pull the image, and runs it fine
so if it was auth issue or creds issue, i should not be able to pull the image in the first place,
anyways i tried to pass
{"auths":{"https://index.docker.io/v2/":{"username":"me","password":"pass","auth":"TOKEN"}}}
and also tried to change the api version to v1 , still gives me the same output on the secrets webhook, and still able to pull without any issues

@hacktron95 Thanks for your answer. We will investigate it because it seems a bug in the webhook.

I just found the bug, PR with incoming.

Duplicate of https://github.com/banzaicloud/bank-vaults/issues/761

Fix in https://github.com/banzaicloud/bank-vaults/pull/779

Hello that is the problem to me.
Describe the bug:
my deployment config
kind: Deployment
metadata:
  name: vault-test
  namespace: prod
spec:
  replicas: 1
  selector:
    matchLabels:
      app.kubernetes.io/name: vault
  template:
    metadata:
      labels:
        app.kubernetes.io/name: vault
      annotations:
        vault.security.banzaicloud.io/vault-addr: "https://vault:8200" # optional, the address of the Vault service, default values is https://vault:8200
        vault.security.banzaicloud.io/vault-role: "apps_prod_roles" # optional, the default value is the name of the ServiceAccount the Pod runs in, in case of Secrets and ConfigMaps it is "default"
        vault.security.banzaicloud.io/vault-skip-verify: "true" # optional, skip TLS verification of the Vault server certificate
        vault.security.banzaicloud.io/vault-path: "kubernetes" # optional, the Kubernetes Auth mount path in Vault the default value is "kubernetes"
    spec:
      serviceAccountName: apps-prod
      containers:
      - name: alpine
        image: alpine
        command: ["sh", "-c", "echo $AWS_SECRET_ACCESS_KEY && echo going to sleep... && sleep 10000"]
        env:
        - name: AWS_SECRET_ACCESS_KEY
          value: vault:secret/data/valami/aws#AWS_SECRET_ACCESS_KEY```

**Env in conteiner**

``` vault-test-565fbfbfc-gc9lr                                        1/1     Running       0          15s
[will@unknown test-projet]$ kubectl logs vault-test-565fbfbfc-gc9lr -n prod
time="2020-02-10T16:40:37Z" level=info msg="Received new Vault token"
time="2020-02-10T16:40:37Z" level=info msg="Initial Vault token arrived"
s3cr3t
going to sleep...
[will@unknown test-projet]$ kubectl exec -ti vault-test-565fbfbfc-gc9lr sh -n prod
/ # env |grep aws
AWS_SECRET_ACCESS_KEY=vault:secret/data/valami/aws#AWS_SECRET_ACCESS_KEY
/ #

OBS: With the command echo return the correct value to env, but inside the pod that value is a string.
Expected behaviour:
i expected vault-webhook to inject the secrets in my container.
Logs webhook
time="2020-02-10T14:43:29Z" level=info msg="found image nginx:stable in cache"
time="2020-02-10T14:50:56Z" level=info msg="found no credentials for registry https://index.docker.io, assuming it is public"
time="2020-02-10T14:50:56Z" level=info msg="I'm using registry https://index.docker.io"
2020/02/10 14:50:56 registry.manifest.get url=https://index.docker.io/v2/library/alpine/manifests/latest repository=library/alpine reference=latest
2020/02/10 14:50:56 registry.blob.download url=https://index.docker.io/v2/library/alpine/blobs/sha256:e7d92cdc71feacf90708cb59182d0df1b911f8ae022d29e8e95d75ca6a99776a repository=library/alpine digest=sha256:e7d92cdc71feacf90708cb59182d0df1b911f8ae022d29e8e95d75ca6a99776a
time="2020-02-10T14:53:41Z" level=info msg="found no credentials for registry https://index.docker.io, assuming it is public"
time="2020-02-10T14:53:41Z" level=info msg="I'm using registry https://index.docker.io"
2020/02/10 14:53:41 registry.manifest.get url=https://index.docker.io/v2/library/alpine/manifests/latest repository=library/alpine reference=latest
2020/02/10 14:53:41 registry.blob.download url=https://index.docker.io/v2/library/alpine/blobs/sha256:e7d92cdc71feacf90708cb59182d0df1b911f8ae022d29e8e95d75ca6a99776a repository=library/alpine digest=sha256:e7d92cdc71feacf90708cb59182d0df1b911f8ae022d29e8e95d75ca6a99776a

Environment details:
Kubernetes version: v1.14.8-gke.33
on prem cluster
vault-secrets-webhook chart- v0.8.0
/kind bug


installation of 1.4

Hi,
I was trying to install CITE-seq-Count v1.4. To perform clean-installation, I uninstalled CITE-seq-Count by using 'pip uninstall' command and reinstalled CITE-seq-Count by using 'pip install CITE-seq-Count'.
After installation, it seems that v1.3.2 was installed instead of v1.4. Do you have any suggestions?
Thank you.
Joon

Hello, I would like to know if the bug was only in version 1.3.4 or also in previous versions.
Thank you very much
Alice

Hello,
it was only in 1.3.4. The versions before had non corrected UMI counts.
Can I close this?

OK, thank you very much.


ci(github): add labeler config to automatically label pull-requests

Goal
Automatically add labels to PRs, so that reviewers have an easier time in navigating the ones they should be interested in.
Description
This PR introduce a configuration that trigger a Github feature for automatically adding labels on PRs, depending on which files have been modified within the PR.
The feature is describe here: https://github.com/actions/labeler
Pull Request Type

[ ] Documentation content changes
[ ] Bugfix / Missing Redirects
[X] Docusaurus site code changes

PR Checklist

[X] The commit message follows our guidelines included in the CONTRIBUTING.md
[ ] All tests of Lint, cspell and check-content pass. (How to launch them?)
[X] No sensitive content has been committed

nice, did you check all the labels are correct? You configured all of them using kebab-case but we also have labels with a human readable text (I think we should converge on a single format also @akelity what do you think?)

Nice!!
Did you check all the labels are correct? You configured all of them using kebab-case but we also have labels with a human readable text (I think we should converge on a single format also @akelity what do you think?)

Yes, it's better to choose a unique format. Perhaps for the simplicity of the action, I would leave this as a label (kebab-case) and modify the ones readable by humans.

Should I proceed with renaming labels into kebab-case?
Also, currently I inserted only the main labels regarding the products. If finer classification is needed, feel free to update the list with the correct documentation path.

Should I proceed with renaming labels into kebab-case?

I think @akelity is fine with the current naming

Also, currently I inserted only the main labels regarding the products. If finer classification is needed, feel free to update the list with the correct documentation path.

We can add more later once we have it running :)


Config file ignored

Hi,
I have installed<EMAIL_ADDRESS>on my MacBook. I want to run static analysis on code in a project, but not on any tests, documentation or Storybook stories, etc. that are placed near source code.
Here is my fta.json file that is completely ignored by fta running from npm script in package.json, npm run --config-path="./fta.json":
{
  "output_limit": 5000,
  "score_cap": 1000,
  "include_comments": false,
  "exclude_under": 6,
  "exclude_directories": [],
  "exclude_filenames": [
    ".spec.jsx",
    ".stories.tsx",
    ".docs.mdx",
    ".types.ts",
    ".d.ts"
  ],
  "extensions": [
    ".js",
    ".jsx",
    ".ts",
    ".tsx"
  ]
}

My fta.json is located in the root of the project and according to documentation it should be picked up automatically, but it does not. Specifying --config-path does not help. I want to ignore any .spec, .docs or. stories, or .types files included in src but this is the output I get:

I tried changing other configuration in the file, like output_limit but they get ignored, too.

Hi @zettadam thanks for reporting. This might be due to some quirkiness of how the exclude/extensions options work.
Do you still have the problem if you clear out extensions? It seems like something is overriding your exclusions. You could also try values like *.spec.jsx (including the asterisks) to see if it changes the result.


Properly document deprecation of DH_new() and related functions

Checklist

[x] documentation is added or updated

This pull request is ready to merge

Merged to master and 3.0. Thank you.


Is it possible to enable Secure Boot and Flash Encryption for ESP32 using AWS FreeRTOS project?

In espressif's esp-idf documentation for Flash Encryption there is mention of a "Security Features" menu option that should come up when running .\vendors\espressif\esp-idf\tools\idf.py menuconfig but I don't see it.
And in the documentation for Secure Boot it mentions that there should be a "Secure Boot Configuration" menu after running .\vendors\espressif\esp-idf\tools\idf.py menuconfig but I don't see that either.
Is there any other way to configure and enable the Flash Encryption and Secure Boot features for my esp32 based project or will I have to take a specific version of esp-idf with these features already configured and enabled and link it into my aws freertos project?

Hi @octopusbunch,
I'm looking into this issue and will get back to you soon.

Hi @shubhamkulkarni97,
Thank you, I look forward to any feedback or help you can give me.

Hi @octopusbunch,
I have created a PR to enable security features with CMake. Please try the changes and let me know if it works for you.
Thanks,
Shubham

Hi @shubhamkulkarni97,
Thank you for the fix, I can see the "Security Features" options show up now with the menuconfig.
But I'm wondering what is the correct version of the esp-idf documentation to reference for the esp-afr-sdk submodule in amazon freertos? I had been using version 3.3.2, but I see there is differences between what that documentation says and the actual menuconfig options. Thank you.

@octopusbunch, esp-afr-sdk submodule is based on ESP-IDF v3.3.2.
What differences did you notice in the documentation and menuconfig?

Thanks @shubhamkulkarni97,
Thank you for the confirmation. In the Secure Boot documentation for v3.3.2 the instructions say to enable Secure Boot under Bootloader Config menu after running menuconfig, but those options for Secure Boot seem to be under Security Features now. "Run make menuconfig, navigate to “Secure Boot Configuration” and select the option “One-time Flash”.
Since esp-afr-sdk is based on esp-idf v3.3.2, does that mean that Secure Boot V2 is not supported with esp-afr-sdk? I would like to take advantage of that feature since I am using an ESP32 V3 for my application.

@octopusbunch,

Thank you for the confirmation. In the Secure Boot documentation for v3.3.2 the instructions say to enable Secure Boot under Bootloader Config menu after running menuconfig, but those options for Secure Boot seem to be under Security Features now. "Run make menuconfig, navigate to “Secure Boot Configuration” and select the option “One-time Flash”.

Documentation states that Pay particular attention to the “Bootloader Config” options, as you can only flash the bootloader once. Because in One-time Flash mode you cannot flash bootloader again.
It doesn't state that these options are under Bootloader Config.

Since esp-afr-sdk is based on esp-idf v3.3.2, does that mean that Secure Boot V2 is not supported with esp-afr-sdk? I would like to take advantage of that feature since I am using an ESP32 V3 for my application.

Yes, Secure Boot V2 is not supported in esp-afr-sdk currently.
However, we already have a PR to update esp-afr-sdk to ESP-IDF v4.2 release. You can try Secure Boot V2 in PR https://github.com/aws/amazon-freertos/pull/2893
Hope this helps!

Hi @shubhamkulkarni97,
Sorry I made a mistake on my earlier post, let me correct my response. Step 1 under How to Enable Secure Boot in docs v3.3.2 reads Run make menuconfig, navigate to “Secure Boot Configuration” and select the option “One-time Flash”. (To understand the alternative “Reflashable” choice, see Re-Flashable Software Bootloader.)
But I didn't see a "Secure Boot Configuration" option in the main menu of menuconfig, it looks like the options for Secure Boot are under "Security features" instead. It's not really a big deal though because I was still able to find it.
Anyways , thanks for the PR. I will try using that version instead for Secure Boot V2.

The issue looks resolved by PR https://github.com/aws/amazon-freertos/pull/2897 which is a fix for ESP-IDF v3.3.
For the Secure Boot V2, it will be available in ESP-IDF v4.2 once PR #2893 get merged.


[json toc] nested sub-folder paths are not being added to html paths

Nested sub-folder paths are not being added to html paths. They are there in xml, but not in json.
Sample output:
https://github.ibm.com/Bluemix-Docs/docs-build-markdown-output2/tree/master/overview
xml:  <topic href="whats-new/index.html#whatsnew" label="What's new in IBM Cloud">
json:
{
    "href": "index.html#whatsnew",
    "label": "What's new in IBM Cloud",
}

Fixed, closing.


Installation Instructions Doesn't Find the Latest Release on Linux

Environment
Device and OS: Linux
App version:
Kubernetes distro being used:
Other:
Steps to reproduce

Linux Installation Instructions under /getting-started/install includes two curl commands.  The first curl fails to produce the expected output, which causes the second curl to fail to download zarf.

Expected result
The first curl command should return the latest version so that the second curl command can download zarf.
Actual Result
The first curl command doesn't follow redirects, so it's not handling the correct URL.
Visual Proof (screenshots, videos, text, etc)
Severity/Priority
Additional Context
Suggested change From:
ZARF_VERSION=$(curl -sIX HEAD https://github.com/defenseunicorns/zarf/releases/latest | grep -i ^location: | grep -Eo 'v[0-9]+.[0-9]+.[0-9]+')
To:
ZARF_VERSION=$(curl -o /dev/null -sLw %{url_effective} https://github.com/defenseunicorns/zarf/releases/latest | awk -F/ '{print $NF}')
The above change to the first curl command will allow the second curl command (the zarf download) to succeed.

Thanks for the issue! Docs site is fixed


Add access log for rest and router connections

Add access which logs authenticated requests to controller with:

timestamp
IP Address
identity id
resource (path)
access type (REST)

Also log router connections, include:

timestamp
IP Address
router id
access type (router controller channel)

Any guidance on when this feature will be added?

Any guidance on when this feature will be added?

I'll see if I can get it in the next release with HA Beta 1.


feat: make oidc settings managable

updates zitadel-go
tidies go modules
adds resource and datasource zitadel_default_oidc_settings
tests the resource
adds examples and doc templates
adds a convenience container that creates a human admin user in the acceptance environment

@stebenz the docs generation with tfplugindocs somehow created two files per resource and datasource each. One with the .md file extension, the other without any file extension. Can you please try to generate the docs?

@stebenz would you mind generating the docs? See PR description.

@eliobischof for next time, the template has to have the .md.tmpl file ending


src/http/ilogger.cpp:376:53: error: ‘bind’ is not a member of ‘std’                  flush_thread_.reset(new thread(std::bind(&Logger::flush_job, this)));

src/http/ilogger.cpp:376:53: note: ‘std::bind’ is defined in header ‘’; did you forget to ‘#include ’?
src/http/ilogger.cpp:38:1:
+#include
大佬在restful_server文件夹下src/http/ilogger.cpp添加一下#include

我加我加

加了


fix(scripts): fix version script

Version isn't updated anymore since templates were moved to root of the project

Follow-up of #2189
CC/ @mcansh

rats! thanks @MichaelDeBoey these should actually be "*" so they can get replaced during create-remix

@mcansh I'll update the package.jsons & revert the version script changes then 👍

@mcansh I'll update the package.jsons & revert the version script changes then 👍

perfect, thanks!

aside, but the best part of these no longer being bundled is there doesn't need to be a release!

Less work for us/our scripts 🥳🎉


Issue with printing a page containing an accordion

When I print a page containing a Bootstrap accordion in Chrome, a blank page is added to the end of the printout.
As part of my investigation in to this issue, I navigated to the accordion example on the Bootstrap website in Chrome (http://getbootstrap.com/javascript/) and entered print preview mode where I was able to also see the issue. I don't see this issue in IE or Firefox within either my application or the Bootstrap website. If you access other pages on the Bootstrap website, without the accordion, there are no blank pages at the end of the printout (see: http://getbootstrap.com/components/, http://getbootstrap.com/css/, http://getbootstrap.com/customize/, http://getbootstrap.com/getting-started/).
Not sure whether this is a Bootstrap or Chrome issue. Could it be looked at please?
Chrome version: Version 59.0.3071.115 (Official Build) (64-bit)
Bootstrap version: v3.3.7
Thanks.

Bootstrap 3 is essentially in maintenance mode as we focus on working towards a stable v4. As such, we're only accepting changes to v3's code on a case-by-case basis, usually only for critical bug fixes or docs improvements.


Redis embedding store does not work anymore

The error is:
{"code":-32603,"message":"Method [io.quarkiverse.langchain4j.quarkus-langchain4j-core.findRelevant] failed: Syntax error at offset 1 near >["}

The passed query is `*=>[ KNN 10 @vector $BLOB AS vector_score ]`

The ingestor is just:
    public void ingest(@Observes StartupEvent ev,
                       RedisDataSource redis,
                       RedisEmbeddingStore store, BgeSmallEnQuantizedEmbeddingModel embeddingModel,
                       @ConfigProperty(name = "rag.location") Path documents) {
        redis.flushall(); // cleanup the Redis to start fresh (for demo purpose only)
        List<Document> list = FileSystemDocumentLoader.loadDocumentsRecursively(documents);
        EmbeddingStoreIngestor ingestor = EmbeddingStoreIngestor.builder()
                .embeddingStore(store)
                .embeddingModel(embeddingModel)
                .documentSplitter(recursive(100, 25))
                .build();
        ingestor.ingest(list);
        Log.info("Documents ingested successfully");
    }

wondering is the syntax should not be (*)=>

cc @jmartisk

*=> and (*)=> should be equivalent, that is the metadata filter, and we send * when there's no filter in use
I tried and couldn't reproduce it (I'm using the chatbot sample, which has pretty much the same code as you pasted in the description, and with redis/redis-stack:latest image)

So most likely it sounds to me like an incompatible Redis version?!

@jmartisk Can you check the sha of your Redis stack image. Here is mine:
docker.io/redis/redis-stack                                                                         latest               ce0b378f9900  7 weeks ago    748 MB

@jmartisk Sorry, forgot to post the application I'm using.
It's https://github.com/cescoffier/quarkus-langchain4j-workshop/tree/main/step-06.
Just replace the PGVector one with the Redis one.

Looking at it - your app starts working for me when I switch it to Quarkus 3.12.1 instead of 3.14.4. So I think the difference is something that has changed in the Redis client extension on the Quarkus side.

I can probably fix that, but we might wanna consider adding some sort of CI against a matrix of supported Quarkus versions. AFAIK we only have something with the latest Quarkus snapshot and it is broken now. That's how we missed that Quarkus 3.13 breaks this embedding store.

We're in a bit of a pickle, because changing to .dialect(2) doesn't work on Quarkus < 3.13. We would have to detect the running Quarkus version and decide what to do based on that....
The other option is to just deal with it that right now it will not work on 3.13+, but when we upgrade to 3.15, we make the fix along with that.

I would much rather have a temporary fix for now (Quarkus LangChain4j 0.18) and then remove it for 0.19

Ok but to have a temporary fix that works on all Quarkus versions from 3.8 to 3.14, we will have to detect the Quarkus version and choose which method to call based on that, is that doable?

Detecting the version is easy :).
As for the which method to call, what exactly are you referring to?

Any example where else we do this?
https://github.com/quarkiverse/quarkus-langchain4j/blob/0.18.0.CR1/embedding-stores/redis/runtime/src/main/java/io/quarkiverse/langchain4j/redis/RedisEmbeddingStore.java#L177
here we need to call .param("DIALECT", "2") on Quarkus <= 3.12 and replace with .dialect(2) on Quarkus >= 3.13

Oh, okay :).
io.quarkus.builder.Version.getVersion() will give you the Quarkus version in a Quarkus build step.
As the bean is synthetic, you can just pass that version to the recorder and use it at will


[Android] No such file or directory

Current behavior
hello, i'm facing issue running android on monorepo project, can't run file because of wrong path of node_modules
i have node_modules in monorepo root only, so is there a way to change the node_modules path on android
/bin/sh: PROJECT_ROOT/apps/rider/node_modules/react-native-keys/keysAndroid.js: No such file or directory

Thanks
Expected behavior
building and running android
Platform

[X] Android
[ ] iOS

React Native Version
0.75.1
React Native Compressor Version
Reproducible Steps And Demo

@HamoBoker did you find any solution for this?


seeing data in postgresql bytea column

in the past, i could right click on a bytea column (containing xml) and select view or edit to see the data.
now i have to copy and paste the data into another app for viewing.

Please try to use Value panel for editing https://github.com/dbeaver/dbeaver/wiki/Panels#value-viewer

Please try to use Value panel for editing https://github.com/dbeaver/dbeaver/wiki/Panels#value-viewer

there is no update on the issue for a long time.
if it is still actual for you - feel free to reopen it with an additional info


Warnings on outputs

When running this with TF 0.11.x these warnings are shown:
Warning: output "asg_name": must use splat syntax to access aws_autoscaling_group.autoscaling_group attribute "name", because it has "count" set; use aws_autoscaling_group.autoscaling_group.*.name to obtain a list of the attributes across all instances
Warning: output "cluster_size": must use splat syntax to access aws_autoscaling_group.autoscaling_group attribute "desired_capacity", because it has "count" set; use aws_autoscaling_group.autoscaling_group.*.desired_capacity to obtain a list of the attributes across all instances
Warning: output "launch_config_name": must use splat syntax to access aws_launch_configuration.launch_configuration attribute "name", because it has "count" set; use aws_launch_configuration.launch_configuration.*.name to obtain a list of the attributes across all instances

Hm, I could've sworn we fixed these when we did our Terraform 0.11 upgrade. 😕
Fairly easy tweak if anyone has time for a PR.


Adds Kana V2 support

Thank you for the PR, the Kana V2 support is now available in rivalcfg 3.4.0 :)
→ https://twitter.com/flogisoft/status/1079063912111394824


shielded-pool: check balance commitment for dummy spends

Is your feature request related to a problem? Please describe.
Similar to #3859. The bug is that the balance commitment check for dummy notes is skipped, so you can put anything you want it and then create a corresponding Output action that will net out to zero (on paper).

This isn't just similar to, but is exactly the same bug as that issue. Closing as duplicate.


-s should log on stdout

cf. https://github.com/crowdsecurity/cs-cloudflare-bouncer/issues/25

Fixed by https://github.com/crowdsecurity/cs-cloudflare-bouncer/pull/27


fails to build for win64-vs2013-debug

The terminal error message:
=== building: win64-vs2013-debug
=== generating: win64-vs2013-debug
CMake Error: Could not create named generator Visual Studio 12 Win64
[ERROR] Failed to generate 'win64-vs2013-debug' of project 'oryol'

Strange, can you try to invoke cmake manually? Start in the oryol directory, and then:
> cd ..
> mkdir build
> cd build
> cmake -G "Visual Studio 12 Win64" ..\oryol
-- The C compiler identification is MSVC 18.0.40629.0
-- The CXX compiler identification is MSVC 18.0.40629.0
...

Also, can you check what cmake version you have (cmake --version), I'm at 3.2.1, but anything above 2.8.11 should work.

PS: What Windows version command line shell do you run? I have tested on Windows7 in the standard 'DOS shell', Cmder and Powershell, and on Windows10 in Cmder (can't test other shells in Windows10 right now because I'm at work).

I used Git Bash terminal and also Cygwin.
On Aug 31, 2015 9:39 AM, "Andre Weissflog"<EMAIL_ADDRESS>wrote:

PS: What Windows version command line shell do you run? I have tested on
Windows7 in the standard 'DOS shell', Cmder and Powershell, and on
Windows10 in Cmder (can't test other shells in Windows10 right now because
I'm at work).
—
Reply to this email directly or view it on GitHub
https://github.com/floooh/oryol/issues/168#issuecomment-136280342.

Hmm ok, I don't run Cygwin, but the normal Git Bash on Windows7 seems to work, so it's probably the Cygwin environment.
Does the manual cmake invocation work? If yes I have a suspicion what it could be: fips is running cmake with the "shell=True" subprocess.call() flag: https://github.com/floooh/fips/blob/master/mod/tools/cmake.py#L55
Setting or not setting this flag produced the type of error you see (unfortunately different on every platforms and shell). Not sure if I want to fix this yet, I would have to add a hack specifically for Cygwin on Windows, and I don't know how I would find out in python that I'm running in a Cygwin environment.


Luca QR codes Not valid

Hi there,
Until now ist is Not possible to  use the check-in function with a Luca based QR code.
Is this a bug (most likely not, as i can not any documentation), or is there a plan to get this feature working?
BR
Detlef

@dherrman1
I suggest you follow the progress of the issue https://github.com/corona-warn-app/cwa-documentation/issues/590 "Link structure for other contact tracing apps" which discusses the Luca app.

@dherrman1 Thanks for the question. This is also discussed, as @MikeMcC399 already mentioned, in #590. We suggest therefore to continue discussion there and close this issue.

Thanks MikeMcC299.

Closed, Issue is described in #590

@dherrman1 FYI https://www.coronawarn.app/de/faq/#check_in_luca


Laravel 9 Support

Hi,
Laravel 9 will be out in a few days so can you add |^9 to composer.json constraints?
Thx!

Done! Cheers @rcerljenko 👍

np  :)


Auto-export stuck on "pending"

Report ID: 9NQF7HP6
It used to work fine and I don't know what changed.
When adding a new entry to my collection, the defined .bib file does not update. If I go to Preferences -> Better BibTeX I see that (1) Automatic Export On Change is marked and (2) the status of my collection is "pending". if I click on the "Export" button on the bottom it updates instantly, but I have to do it again for each entry I add. The automatic export always gets stuck on "pending".
Any ideas on what could have gone wrong?

After looking at the instructions for use again, I see you are not supposed to run Zotero standalone in parallel to the Firefox extension, which I did the day I posted the bug report. I can't reproduce the issue now though.

Yeah, that would do it. Not sure how I'm going to tackle that yet.

Hello, just popping in to say that I have the same issue (stuck on pending). I use the stand-alone Zotero with the Chrome add-on. I'm using the last release (1.6.45). Not sure whether I should open a new issue?

Please do open a new issue.


feature: Support an environment variable for the --full-refresh option of meltano run and meltano el|elt

Feature scope
CLI (options, error messages, logging, etc.)
Description
Many users' deployments generate or hard-code a Meltano command to run on their own infra, and modifying this command for a one-off full refresh of the data is inconvenient.
An environment variable would make triggering a full refresh in such environments considerably easier.
https://github.com/meltano/meltano/blob/b8d68748075ec8e5e464da79a694164d0c62eeca/src/meltano/cli/run.py#L59-L66
https://github.com/meltano/meltano/blob/b8d68748075ec8e5e464da79a694164d0c62eeca/src/meltano/cli/elt.py#L58-L62

https://click.palletsprojects.com/en/8.1.x/api/#click.Parameter

Slack
https://meltano.slack.com/archives/C069CQNHDNF/p1727713013315249?thread_ts=1727710811.205209&cid=C069CQNHDNF

👍

Hi @edgarrmondragon.. I noticed this issue and would like to contribute. Is anyone currently working on it? If not, could you please assign it to me?

@MutuT2 Go for it!


fix(config): add missing parameters to Zooz ZEN32

This adds two parameters:

ability to disable programming function from the button (Parameter 22)
control LED indicator flashing during a setting change (Parameter 23)

The official documentation on Parameter 23 is incorrect. It accepts values 0 (Enabled, default) and 1 (Disabled).
Verified this on my ZEN32 devices.

I got carried away and did a complete rehaul of the Zooz subfolder to use templates wherever this was feasible. Should I close this one and open a new PR for that?

Yes, we appreciate the help.


Setting some 1 sided styling props on TextInput does not work correctly. An update since issue #7070.

Description
This is a bug-update report, following a previously opened issue.
React Native's TextInput docs present a warning on the style prop indicating an incomplete list of 1 sided stylings that will not work on TextInput. They still reference this old 2016 thread: issue #7070 which was simply locked with this warning on the docs, and no real resolution.
I am posting such that, hopefully, some RN devs can look again into the issue, and maybe give us a clear understanding on what works, what doesn't - and maybe the TextInput style warning can be given more info as well...?
To help out, I have tested this myself. My methodology is by far not the best, but I think it covers some of the most frequent use cases. If it is of any relevance, I am testing trough Expo (SDK 40).
All presented issues can be reproduced on Snack.
React Native version:
% npx react-native info
info Fetching system and libraries information...
System:
    OS: macOS 11.1
    CPU: (8) x64 Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz
    Memory: 266.61 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 12.18.2 - /usr/local/bin/node
    Yarn: 1.22.5 - /usr/local/bin/yarn
    npm: 6.14.5 - /usr/local/bin/npm
    Watchman: Not Found
  Managers:
    CocoaPods: Not Found
  SDKs:
    iOS SDK:
      Platforms: iOS 14.3, DriverKit 20.2, macOS 11.1, tvOS 14.3, watchOS 7.2
    Android SDK: Not Found
  IDEs:
    Android Studio: 4.0 AI-193.69<IP_ADDRESS>14223
    Xcode: 12.3/12C33 - /usr/bin/xcodebuild
  Languages:
    Java: 14.0.1 - /usr/bin/javac
    Python: 2.7.16 - /usr/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: 16.13.1 => 16.13.1
    react-native: https://github.com/expo/react-native/archive/sdk-40.0.1.tar.gz => 0.63.2
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps To Reproduce
Start a new blank project, import TextInput, and apply any combination of the styling props described in the list below. Test on iOS and Android at the same time.
List of tested styling props:

1 sided border width
borderTopWidth, borderBottomWidth, borderLeftWidth, borderRightWidth
1 sided border colors
borderTopColor, borderBottomColor, borderLeftColor, borderRightColor
1 sided (cornered) border radiuses
borderTopLeftRadius, borderTopRightRadius, borderBottomLeftRadius, borderBottomRightRadius
1 sided margins
marginTop, marginBottom, marginLeft, marginRight
1 sided paddings
paddingTop, paddingBottom, paddingLeft, paddingRight
shadows
elevation, shadowOffset, shadowOpacity, shadowRadius

Expected Results
As of the time of this post and react-native version 0.63.2, only the following (out of the tested) styling props seem NOT to work as intended.
All issues are only present on Android, iOS works correctly in all (tested) cases.

borderLeftColor and borderRightColor
These styling props work correctly as long as there is no borderColor styling prop. Doesn't matter the order in which they appear in the styling object(s), borderColor will override their values.

All 1 sided border width styling props: borderTopWidth, borderBottomWidth, borderLeftWidth, borderRightWidth
Borders should never overlap with the content area, or the padding area. See the W3schools CSS Box Model for reference.
However, all 1 sided border props will draw within the content box of the TextInput, with borders being visible under the text. They also affect the 1 sided paddings, effectively cancelling out their respective paddingXxx prop
The all sided padding prop seems NOT to be affected by this.

All other tested 1 sided styling props seem to work fine both on iOS and Android, including shadow/elevation, single-line and multi-line input variants. See the snack and screenshots in further sections.
Snack, code example, screenshot, or link to a repository:
Snack
https://snack.expo.io/@cristian-nxtl/textinput---testing-which-1-sided-styling-props-are-not-working
Screenshots
Screenshots have been taken on Simulator/Emulator for convenience, tested on real devices as well, see below.
iOS Simulator: iPhone 11 Pro Max | iOS 14.2
Tested on real device: iPhone SE 1st gen | iOS 14.3

Android Emulator: Pixel 3A | Android 10
Tested on real device: Samsung Galaxy A5 2016 (SM-A510F) | Android 10 (Lineage OS 17.1)

Do you still experience this issue?
I have four years of experience maintaining facebook/react-native and I specialize in the Text and TextInput components. I currently have 58 facebook/react-native PRs.
If you still experience this issue, I will prepare a patched release with the fix.
Thanks a lot


Items shown and total items under view more button

I'm implementing an infinite scroll at a prestashop for products list and I would like to show the items shown and the total items. I have a smarty variable with the total items and the items shown in first place. I'm using the "view more" button option and the problem is that, once the site is loaded, I can't change the smarty variables, I don't know how to do it with javascript and I think there's no infinite-scroll option for that, is it?
{$listing.pagination['total_items']}

{$listing.pagination['items_shown_to']}

<button class="infinite-scroll-button" onclick=""> {* Can't increase items_shown_to at onclick becaus it's a smarty variable *}
    {l s='See more' d='Shop.Theme.Actions'}
</button>

{l s='You have seen' d='Shop.Theme.Global'} {$items_shown_to} {l s='products of' d='Shop.Theme.Global'} {$listing.pagination['total_items']}

Test case: https://codepen.io/miguel-gisbert/pen/QWBXjBd

I think you should choose another library as this seems a legacy project already

Thanks, but I can't change the library. That's the one they want and they use for all the sites.

I finally sorted it out at the js event that shows more products when the button is clicked (append.infiniteScroll) which actually has the number of new items that have been displayed on each click (items.length). I have all this at themes/my_theme/templates/catalog/_partials/productlist.tpl
I've just used a span at the products.tpl with an attribute to get the current items shown.
<button class="infinite-scroll-button">
    {l s='See more' d='Shop.Theme.Actions'}
  </button>

{l s='Has visto' d='Shop.Theme.Global'} {$listing.pagination['items_shown_to']}
   {l s='productos de' d='Shop.Theme.Global'} {$listing.pagination['total_items']}

And I update the value at the append event:
$('#items_shown_to').attr('items_shown',parseInt($('#items_shown_to').attr('items_shown'))+parseInt(items.length));
        $('#items_shown_to').text($('#items_shown_to').attr('items_shown'));

I hope this is useful for someone else.
Thanks.


why is there a white bar on the left of generated images?

"Fixed" this by not subtracting the vggnet mean pixel.


Update old #index-entry-... links

Amend links with stale #index-entry... targets, in some cases choosing page sections rather than the equivalent index entry.
Manually tested updated links locally.

Do we know why this were changed originally?

@cfa I have been working hard on this issue. I have now determined that X<> markup in =head contexts was rendered into HTML differently to X<> in ambient text by Documentable / Pod::To::HTML. I am currently working on some of this.

What's the current status on this, @finanalyst ? Is there a ticket open tracking the related work you mention?

@coke Sorry for the considerable delay. I have been working on this issue which is also linked to other problems. Complications include

Targets (eg. index-entry-XXX) were generated by Documentable in different ways depending on whether the X<> was included in the title, or was in the text.
Targets with multiple entries (eg L<display text|first entry, some description; second entry, more description> are treated oddly, with the target being index-entry-some_description-more_description-display_text.
Targets with the form L<C<some term>|/routine/filename> exclude the text formatting (eg. some terms from the name of the target
Targets with filenames containing escaped chars included things like $SOLIDUS
Targets in Secondary files have their own naming form
There are errors in the source files completely unrelated to the Documentable naming strategy

All these target names are distributed around Documentable classes, so it has been very difficult to reverse engineer the names.
I think I am making progress, and I hope to create a PR for doc-website tooling in the near future.

@cfa @coke

I have modified the id naming algorithm for the case when a heading contains X<heading text|Syntax,foo>
This eliminates a large number of errors related to index-entry-... type links.
One example is index-entry-qw-quote-words-quote-words is now generated (this was the first change in @cfa PR.

Some general comments about the PR. I think that it should be

There are 15 changes suggested here where one index-entry.... is replaced by another index-entry...... These should now be excluded from the PR.

In general, however, I think that the changes from 'index-entry' to the heading name is better. In many cases the X<> markup is after the heading. Consequently when the browser hits the X anchor, the heading is placed above the browser window. When the heading is targeted the browser puts the heading in a visible position.
The changes to the production web-site have yet to be approved. So the new index-entry names can be seen in new-raku.finanalyst.org

@cfa  - can you rebase this and address @finanalyst's comments? Would love to get it merged finally!

Instead of trying to apply this, will work off the error report referenced in https://github.com/Raku/doc/issues/4476
Thanks, @cfa, for giving this a shot, I'm sorry we didn't get it merged in time. :(


2.0 Error: AttributeError: 'ParentalManyToManyField' object has no attribute 'rel'

Found a bug? Please fill out the sections below. 👍
Issue Summary
When upgrading a package for 2.0, I get the error AttributeError: 'ParentalManyToManyField' object has no attribute 'rel' after having built in try/except block for Wagtail 2.0 imports when invoking runserver.
Steps to Reproduce
Here is the stack trace:
Unhandled exception in thread started by <function check_errors.<locals>.wrapper at 0x7ff4f8382048>
Traceback (most recent call last):
  File "/home/vagrant/.virtualenvs/wagtail2/lib/python3.6/site-packages/django/utils/autoreload.py", line 225, in wrapper
    fn(*args, **kwargs)
  File "/home/vagrant/.virtualenvs/wagtail2/lib/python3.6/site-packages/django/core/management/commands/runserver.py", line 112, in inner_run
    autoreload.raise_last_exception()
  File "/home/vagrant/.virtualenvs/wagtail2/lib/python3.6/site-packages/django/utils/autoreload.py", line 248, in raise_last_exception
    raise _exception[1]
  File "/home/vagrant/.virtualenvs/wagtail2/lib/python3.6/site-packages/django/core/management/__init__.py", line 327, in execute
    autoreload.check_errors(django.setup)()
  File "/home/vagrant/.virtualenvs/wagtail2/lib/python3.6/site-packages/django/utils/autoreload.py", line 225, in wrapper
    fn(*args, **kwargs)
  File "/home/vagrant/.virtualenvs/wagtail2/lib/python3.6/site-packages/django/__init__.py", line 24, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/home/vagrant/.virtualenvs/wagtail2/lib/python3.6/site-packages/django/apps/registry.py", line 112, in populate
    app_config.import_models()
  File "/home/vagrant/.virtualenvs/wagtail2/lib/python3.6/site-packages/django/apps/config.py", line 198, in import_models
    self.models_module = import_module(models_module_name)
  File "/home/vagrant/.virtualenvs/wagtail2/lib64/python3.6/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/vagrant/projects/peregrine/peregrine/models.py", line 50, in <module>
    class SitePage(ContentStreamPage):
  File "/home/vagrant/.virtualenvs/wagtail2/lib/python3.6/site-packages/django/db/models/base.py", line 152, in __new__
    new_class.add_to_class(obj_name, obj)
  File "/home/vagrant/.virtualenvs/wagtail2/lib/python3.6/site-packages/django/db/models/base.py", line 315, in add_to_class
    value.contribute_to_class(cls, name)
  File "/home/vagrant/.virtualenvs/wagtail2/lib/python3.6/site-packages/modelcluster/fields.py", line 484, in contribute_to_class
    setattr(cls, self.name, self.related_accessor_class(self.rel))
AttributeError: 'ParentalManyToManyField' object has no attribute 'rel'

Relevant parts of pip freeze:
Django==2.0
django-modelcluster==3.1
django-taggit==0.22.1
django-treebeard==4.2.0
djangorestframework==3.7.3
-e git+https://github.com/wagtail/wagtail.git@8aa8ddcde986eee80e62ba6f185071e24669aeea#egg=wagtail

The models kicking the error:
class Category(models.Model):
    """
    Categories which a Page or Post item can belong to. A page can belong to
    many categories.
    """
    name = models.CharField(max_length=255)

    class Meta:
        verbose_name_plural = 'categories'

    def __str__(self):
        return self.name

class SitePage(ContentStreamPage):
    header_image = models.ForeignKey(
        'images.Image',
        null=True,
        blank=True,
        on_delete=models.SET_NULL,
        related_name='+',
        help_text='A featured image that will appear in the site theme header.',
    )
    excerpt = RichTextField(
        features=['bold', 'italic', 'link', 'ol', 'ul'],
        blank=True,
        null=True,
        help_text='An short excerpt or abstract about the content.'
    )
    categories = ParentalManyToManyField(
        'Category',
        blank=True,
        help_text='The categories for the page or post.',
    )
    show_in_menus_default = True

    search_fields = ContentStreamPage.search_fields + [
        index.SearchField('body'),
        index.SearchField('excerpt'),
    ]

    content_panels = [
        FieldPanel('title'),
        ImageChooserPanel('header_image'),
        FieldPanel('excerpt'),
        FieldPanel('categories', widget=CheckboxSelectMultiple),
        StreamFieldPanel('body'),
    ]

    class Meta:
        verbose_name = 'Page'

Technical details

Python version: 3.6.3
Django version: 2.0
Wagtail version: 2.0a0
Browser version: Chrome 62.

Hi @FlipperPA,
This error is coming from django-modelcluster, which doesn't yet have a Django 2.0 compatible release - hopefully I'll get one released in the next week, but for now you'll need to use the git master.
django-taggit is also awaiting a Django 2.0 compatible release (for now you'll need to use https://github.com/alex/django-taggit/pull/509), although I'm not sure how much that's a showstopper for Wagtail's purposes.

n00b question:  how do i get   django-modelcluster to master branch?
tried:
pip install git+https://github.com/wagtail/django-modelcluster.git@master
did   pip uninstall django-modelcluster first.
(even after install wheel, it reverts back to 5.1)
adding django-modelcluster>5.1 to requirements.txt didn't help either.
tnx for help !   searching on, meanwhile

Hi @peeterss - please use the Wagtail tag on Stack Overflow for support queries, or see the Support page in the documentation for other options.
(Hopefully you did notice that this is a three year old issue, though...? The fixes mentioned here have been in a proper release for a long time.)


make reference to the adaptor documentation

at xenon/build/docs/javadoc-devel/index.html
specifically, with regard to the properties, and explain that there are different kinds of properties

point to xenon/build/docs/javadoc/index.html or http://nlesc.github.io/Xenon/versions/1.1.0/javadoc/ . Javadoc-devel is for Xenon developers not for developers using Xenon library.

You're right of course.


Add API-Readiness-Checklist.md to RM project

What type of PR is this?

documentation

What this PR does / why we need it:
This file replaces the file in Commonalities and is updated as defined in the API release process in the Release Management Wiki Wiki (see https://wiki.camaraproject.org/display/CAM/API+Release+Process#APIReleaseProcess-APIreadinesschecklist).
Which issue(s) this PR fixes:
Fixes #28
Special notes for reviewers:
Changelog input
 Creation of updated API readiness checklist, removed from Commonalities

Additional documentation

Is it possible to put a link to the related documentation in the Comments column. This would improve the table also providing an overall a guide to the API development. Maybe we could first approve the table and then improve it with this links

@tanjadegroot  I think Fabrizio wanted to add links to guidelines/templates of specific assets, but providing the links to artifacts related to given API version is good idea for me.
@FabrizioMoggio The related documentation for different assets is linked in the Explanations table in the Wiki: https://wiki.camaraproject.org/display/CAM/API+Release+Process#APIReleaseProcess-APIreadinesschecklist

Is it possible to put a link to the related documentation in the Comments column. This would improve the table also providing an overall a guide to the API development. Maybe we could first approve the table and then improve it with this links

@tanjadegroot I think Fabrizio wanted to add links to guidelines/templates of specific assets, but providing the links to artifacts related to given API version is good idea for me. @FabrizioMoggio The related documentation for different assets is linked in the Explanations table in the Wiki: https://wiki.camaraproject.org/display/CAM/API+Release+Process#APIReleaseProcess-APIreadinesschecklist

The link is already there in the last line of the file

@soadeyemo Hi Samuel, can you please review and approve the PR ?

@soadeyemo - Hi Samuel, sorry,  but can you review and approve again please, I fixed some typos and explanations in the checklist.


NuGetPublisher enhancement

Nuget 3.3 introduced an improved folder structure that can be published to using a similar command as push except they called it Add.  Is there a chance this can be included, specifically for On Prem builds.

With TFS vNext we're introducing package management (already shipped in VSTS). That removes the need to deal with fileshares for NuGet. I'm not very inclined to take work that moves the task back towards fileshares.

Understandable, but our company use the tasks to build (internal build agent/servers) and our applications are not and probably will not use dnx in the near future, we also have an internal company NuGet server and would prefer not to install a full NuGet server.

Understood. See here for information on creating and uploading your own build tasks (potentially forking ours - you can give it a new GUID and it won't be overwritten by future updates).
Package management in TFS won't require you to run your own NuGet server at all. It's built into the product.

As far as I understand, the link you posted and all the instructions for deploying custom tasks are all related to VSO because you can't login to On-Prem TFS with a personal access token.  Therefore, I am unable to upload custom tasks to our On-Prem server.  Or am I missing something?
I'm sorry but I don't understand your final sentence.  Any chance you can shed some light on that?
Thank you for your time so far.

Ah, sorry. I'm very cloud-focused and forgot that on-prem doesn't have PATs until Update 2. @chrisrpatterson how do on-prem customers upload new build tasks?
My final comment was addressing your statement "we prefer not to install a full NuGet server". I work on Package Management which provides a NuGet (and soon, npm) service directly in TFS. There won't be a separate server to set up and manage, it will be a feature of the product.
Feel free to ping me at<EMAIL_ADDRESS>(yes, funny domain because I used to work in Xbox before coming to TFS) if you want to discuss further!


Don't use preferred_username for authZ

Updated description of preferred_username.

@skwan,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@skwan : Thanks for your contribution to the Azure documentation! The author, @dstrockis, has been notified to review your proposed change.

#sign-off


AllJoynExplorer crashes on Windows 10 Mobile

When deploying the alljoyn explorer to Windows 10 mobile, the app will launch but then crash a second later.

FYI @zhuridartem

Noted. We will attempt to repro and get back to this post.

Won't Fix. AllJoyn Explorer is currently not supported on Windows 10 mobile, with no ETA.
Note that DeviceProvider based projects should not be crashing on Mobile, IoT Core, or other editions.

Sad Panda :-(
Quite useful to have a device next to you to monitor alljoyn services while debugging


Add DockerContainerDumper class

ref: https://github.com/biothings/biothings.api/issues/266
Triggers a docker container (typically runs on a different server) to run and generate the output file, and then stop the container.The dumper class will then get the processed file(s) and send it to the Uploader as normally files to datasource folder
This dumper will act following steps:
- Create a temporary docker volume on the remote server
- Start a container and mount remote folder to the temporary volume
- Wait for the container is stop - Supposing the remote file is ready for download
- Download remote file via Docker API
- Remove above container and volume after successfully download. If it failed to download, the container and volume will not be removed, then you can troubleshot issues by show the logs of this container on the remote server
These are supported connection type from the Hub server to the remote Docker host server:

ssh: Prerequisite: the SSH Key-Based Authentication is configured
unix: Local connection
http: Use the insecure HTTP connection over TCP socket
https:  Use secured HTTPS connection using TLS. Prerequisite:

The Docker API on the remote server MUST BE secured with TLS
A TLS key pair is generated on the Hub server and placed inside the same data plugin folder or the data source folder

The data_url should match the following format:
docker+DOCKER_CLIENT_URL?image=DOCKER_IMAGE&tag=TAG&path=/path/to/remote_file&custom_cmd="this is custom command"

Supported params:

image: (Required) the Docker image name
path: (Required) path to the remote file inside the Docker container
tag: (Optional) the image tag
custom_cmd: (Optional) You don't need to fill this param if your docker entrypoint script already write output to the "/path/to/remote_file" file. Or you can override default docker entrypoint with this command with output to the /path/to/remote_file

Examples:
- docker+unix:///var/run/docker.sock?image=IMAGE_NAME&tag=IMAGE_TAG&path=/path/to/remote_file(inside the container)&custom_cmd="run something with output is written to -O /path/to/remote_file (inside the container)"
- docker+ssh://ubuntu@remote-server?image=IMAGE_NAME&tag=IMAGE_TAG&path=/path/to/remote_file(inside the container)&custom_cmd="run something with output is written to -O /path/to/remote_file (inside the container)"
- docker+http://remote-server:2375?image=IMAGE_NAME&tag=IMAGE_TAG&path=/path/to/remote_file(inside the container)&custom_cmd="run something with output is written to -O /path/to/remote_file (inside the container)"
- docker+http://remote-server:2375?image=IMAGE_NAME&tag=IMAGE_TAG&path=/path/to/remote_file(inside the container)&custom_cmd="run something with output is written to -O /path/to/remote_file (inside the container)"
A data plugin example, with TLS config following doc https://www.howtogeek.com/devops/how-to-secure-dockers-tcp-socket-with-tls/
---
version: '0.3'
requires:
- pandas
- numpy
dumper:
  tls_cert_path: client-certificate.pem
  tls_key_path: client-key.pem
  data_url:
  - docker+https://remote-server:2375?image=confluentinc/cp-kafka&tag=7.0.1&path=/tmp/annotations.zip&custom_cmd="/usr/bin/wget https://s3.pgkb.org/data/annotations.zip -O /tmp/annotations.zip"
  uncompress: true
uploader:
  parser: parser:load_annotations
  on_duplicates: ignore

@remoteeng00 can you provide an example of Dumper class using docker container as the source? Don't need to provide it as part of the PR, you can paste it here.

This is an example that I exported from a manifest plugin:
import os

import biothings, config

biothings.config_for_app(config)
from config import DATA_ARCHIVE_ROOT

from biothings.utils.common import uncompressall

import biothings.hub.dataload.dumper

class MydockerDumper(biothings.hub.dataload.dumper.DockerContainerDumper):

    SRC_NAME = "mydocker"
    SRC_ROOT_FOLDER = os.path.join(DATA_ARCHIVE_ROOT, SRC_NAME)
    SCHEDULE = None
    UNCOMPRESS = True
    TLS_CERT_PATH = "client-certificate.pem"
    TLS_KEY_PATH = "client-key.pem"
    SRC_URLS = [
        'docker+https://remote-ip:2375?image=confluentinc/cp-kafka&tag=7.0.1&path=/tmp/annotations.zip&custom_cmd="/usr/bin/wget https://s3.pgkb.org/data/annotations.zip -O /tmp/annotations.zip"'
    ]
    __metadata__ = {"src_meta": {}}

    def post_dump(self, *args, **kwargs):
        if self.__class__.UNCOMPRESS:
            self.logger.info("Uncompress all archive files in '%s'" %
                             self.new_data_folder)
            uncompressall(self.new_data_folder)

Update:

Example config.py

DOCKER_CONFIG = {
        "DOCKER_TLS_NAME": {
            "tls_cert_path": "/path/to/client-certificate.pem",
            "tls_key_path": "/path/to/client-key.pem",
            "client_url": "https://remote_ip:2375"
        },
        "localdocker": {
            "client_url": "unix://var/run/docker.sock"
        },
        "sshdocker": {
            "client_url": "ssh://ubuntu@remote_ip"
        },
}

Example: manifest.yaml:

---
version: '0.3'
name: 'DOCKER'
requires:
- pandas
- numpy
dumper:
  data_url:
  - docker://DOCKER_TLS_NAME?image=confluentinc/cp-kafka&tag=7.0.1&path=/tmp/annotations.zip&custom_cmd="/usr/bin/wget https://s3.pgkb.org/data/annotations.zip -O /tmp/annotations.zip"
  uncompress: true
uploader:
  parser: parser:load_annotations
  on_duplicates: ignore

Example the dumper class:

from urllib3.exceptions import InsecureRequestWarning
from urllib3 import disable_warnings
import biothings, config

biothings.config_for_app(config)
from config import DATA_ARCHIVE_ROOT

from biothings.utils.common import uncompressall

# Disable InsecureRequestWarning: Unverified HTTPS request is being made to host
disable_warnings(InsecureRequestWarning)

import biothings.hub.dataload.dumper

class MydockerDumper(biothings.hub.dataload.dumper.DockerContainerDumper):

    SRC_NAME = "mydocker"
    SRC_ROOT_FOLDER = os.path.join(DATA_ARCHIVE_ROOT, SRC_NAME)
    SCHEDULE = None
    UNCOMPRESS = True
    SRC_URLS = [
        'docker://DOCKER_TLS_NAME?image=confluentinc/cp-kafka&tag=7.0.1&path=/tmp/annotations.zip&custom_cmd="/usr/bin/wget https://s3.pgkb.org/data/annotations.zip -O /tmp/annotations.zip"'
    ]
    __metadata__ = {"src_meta": {}}

    def post_dump(self, *args, **kwargs):
        if self.__class__.UNCOMPRESS:
            self.logger.info("Uncompress all archive files in '%s'" %
                             self.new_data_folder)
            uncompressall(self.new_data_folder)

Update newest document:
Triggers a docker container (typically runs on a different server) to run and generate the output file, and then stop the container.
The dumper class will then get the processed file(s) and send it to the Uploader as normal files to the data source folder
This dumper will do the following steps:
- Start a container if it stopped or create a new container if container_name param is not provided
- Run the get_version_cmd inside this container - if it provided
- Run the custom_cmd inside this container - if it provided
- Download the remote file via Docker API, extract the downloaded .tar file
- Remove the above container and volume after successfully downloading - if keep_container=true
These are supported connection types from the Hub server to the remote Docker host server:
  - ssh: Prerequisite: the SSH Key-Based Authentication is configured
  - unix: Local connection
  - http: Use an insecure HTTP connection over a TCP socket
  - https:  Use a secured HTTPS connection using TLS. Prerequisite:
        - The Docker API on the remote server MUST BE secured with TLS
        - A TLS key pair is generated on the Hub server and placed inside the same data plugin folder or the data source folder

The data_url should match the following format:
    docker://CONNECTION_NAME?image=DOCKER_IMAGE&tag=TAG&path=/path/to/remote_file&custom_cmd="this is custom command"&container_name=CONTAINER_NAME&keep_container=true&get_version_cmd="cmd"

All info about Docker client connection MUST BE defined in the `config.py` file, under the DOCKER_CONFIG key, Ex:
    DOCKER_CONFIG = {
        "CONNECTION_NAME_1": {
            "tls_cert_path": "/path/to/cert.pem",
            "tls_key_path": "/path/to/key.pem",
            "client_url": "https://remote-docker-host:port"
        },
        "CONNECTION_NAME_2": {
            "client_url": "ssh://user@remote-docker-host"
        },
    }

Supported params:
   - image: (Optional) the Docker image name
   - container_name: (Optional) If this param is provided, the `image` param will be discard when dumper run.
   - path: (Required) path to the remote file inside the Docker container
   - tag: (Optional) the image tag
   - custom_cmd: (Optional) You don't need to fill this param if your docker entry point script already writes output to the "/path/to/remote_file" file.
                Or you can override the default docker entry point with this command with output to the /path/to/remote_file
   - keep_container: (Optional) accepted values: true/false, default: false. If keep_container=true, the remote container will be persisted, else if will be remove after successful dump
   - get_version_cmd: (Optional) The custom command for checking release version of local and remote file. Note that:
     - This command must run-able in both local Hub (for checking local file) and remote container (for checking remote file).
     - "{}" must exist in the command, it will be replace by the data file path when dumper runs,
        ex: get_version_cmd="md5sum {} | awk '{ print $1 }'" will be run as: md5sum /path/to/remote_file | awk '{ print $1 }' and /path/to/local_file
Ex:
  - docker://CONNECTION_NAME?image=IMAGE_NAME&tag=IMAGE_TAG&path=/path/to/remote_file(inside the container)&custom_cmd="run something with output is written to -O /path/to/remote_file (inside the container)"
  - docker://CONNECTION_NAME?image=IMAGE_NAME&tag=IMAGE_TAG&path=/path/to/remote_file(inside the container)&custom_cmd="run something with output is written to -O /path/to/remote_file (inside the container)"
  - docker://CONNECTION_NAME?image=IMAGE_NAME&tag=IMAGE_TAG&path=/path/to/remote_file(inside the container)&custom_cmd="run something with output is written to -O /path/to/remote_file (inside the container)"
  - docker://CONNECTION_NAME?image=IMAGE_NAME&tag=IMAGE_TAG&path=/path/to/remote_file(inside the container)&custom_cmd="run something with output is written to -O /path/to/remote_file (inside the container)"
  - docker://CONNECTION_NAME?image=IMAGE_NAME&tag=IMAGE_TAG&path=/path/to/remote_file(inside the container)&custom_cmd="run something with output is written to -O /path/to/remote_file (inside the container)"
  - docker://CONNECTION_NAME?image=IMAGE_NAME&tag=IMAGE_TAG&path=/path/to/remote_file(inside the container)&custom_cmd="run something with output is written to -O /path/to/remote_file (inside the container)"
  - docker://CONNECTION_NAME?container_name=CONTAINER_NAME&path=/path/to/remote_file(inside the container)&custom_cmd="run something with output is written to -O /path/to/remote_file (inside the container)&keep_container=true&get_version_cmd="md5sum {} | awk '{ print $1 }'"
"""

Example about manifest.yaml:
version: '0.3'
name: 'DOCKER'
requires:
- pandas
- numpy
dumper:
  data_url:
  - docker://docker1?container_name=serene_stonebraker&path=/tmp/annotations.zip&custom_cmd="/usr/bin/wget https://s3.pgkb.org/data/annotations.zip -O /tmp/annotations.zip"&keep_container=true&get_version_cmd="md5sum {} | awk '{ print $1 }'"
  uncompress: true
  release: "version:get_release"
uploader:
  parser: parser:load_annotations
  on_duplicates: ignore

For more examples: https://github.com/remoteeng00/plugins/tree/master/mydocker

Update:

Rename custom_cmd to dump_command
Use LABEL in Dockerfile as the container metadata to provide dumper parameters, please check my example: https://github.com/remoteeng00/plugins/tree/dockerfile-as-metadata/mydocker

Update 2023-03-29:

From now, you can custom all biothings.config include the DOCKER_CONFIG attribute by define the config.py file in the working directory.
Example: $ cat config.py

DOCKER_CONFIG = {
    "docker1": {
        "tls_cert_path": "/path/to/client-certificate.pem",
        "tls_key_path": "/path/to/client-key.pem",
        "client_url": 'https://<remote IP>:<remote port>'
    },
    "localhost": {
        "client_url": "unix://var/run/docker.sock"
    }
}
DATA_SRC_DATABASE = 'DATA_SRC_DATABASE'
DATA_SRC_DUMP_COLLECTION = 'data_src_dump_collection'
DATA_HUB_DB_DATABASE = 'data_hub_db_database'
HUB_DB_BACKEND = {
    "module": "biothings.utils.sqlite3",
    "sqlite_db_folder": "./hub_db_folder"
}
DATA_ARCHIVE_ROOT = "./data_archive_root"


Cannot start Meteor using Vagrant image

The following message appears after starting meteor:
vagrant@gazelle:/var/www$ meteor run --settings settings.json
[[[[[ /var/www ]]]]]
=> Started proxy.
=> Meteor 1.2.1: React and Angular support, ECMAScript 2015, new faster build
system, and improvements to the Cordova integration for mobile apps.
This release is being downloaded in the background. Update your app to
Meteor 1.2.1 by running 'meteor update'.
Unexpected mongo exit code 1. Restarting.
Unexpected mongo exit code 1. Restarting.
Unexpected mongo exit code 1. Restarting.
Can't start Mongo server.

According to online documentation, this should resolve itself.
If it does not, reopen this issue.


relocation error when running c3803116bd70e802483d3bc4c4b564d2

./c3803116bd70e802483d3bc4c4b564d2: relocation error: ./c3803116bd70e802483d3bc4c4b564d2: symbol free, version GLIBC_2.2.5 not defined in file libc.so.6 with link time reference

possibly a result of the mmap bugs

Fixed in latest master.


Static content on top of slideshow

Is it possible to add static content on top of the slideshow? I've tried setting it with absolute positioning but can't get it to work. Would like to add a site logo and a main menu, similar to the attachment below. Can it be done?
Many thanks in advance!

you can probably achieve this with absolute positioning of the content.
make sure logo + menu get a higher z-index than the slippry container and position them absolute. position the slippry container absolute and add that all to a container with relative positioning.
then it is only about positioning the elements. all should have a top: 0 and the logo + slippry left: 0, the menu right: 0
I hope this helps, all the best!

Thanks for the quick reply! Got it working now, thought I had it setup just like that before as well but must have missing something.
One issue however: since the slider is now positioned absolutely, if overflows the content below and setting a fixed height isn't really possible. If I understand things correctly, the height is set by the fillerClass but this can't affect the content below anymore. Is there any way to get the calculated top padding to a element outside the slider?

 The slideshow class doesn't have to be positioned absolutely.


Training with mixed precision is returning NaN

Thank you for your great work
When using mixed_precision: True https://github.com/plemeri/InSPyReNet/blob/bfe08191800dc0d7212a931052e8ef4b679d7db7/configs/InSPyReNet_SwinB.yaml#L39, the model is returning NaN
Do you have an idea how to fix this issue?
Thanks

Hi, thank you for your interest in our work. We tried using mixed_precision to speed up training, but ended up with not using it because it resulted much worse performance compared to the default setting. It was used in the early stage of our research, so we recommend not to enable it.
Thank you.

Thank you for your answer.
Any heads up on the performances of the model after conversion into fp16 after training though ?


[AppBar] Port component

[x] Component
[x] Tests (at least unit tests)
[x] Docs
[x] Demo

Started, needs a little more work. @newoga any desire to implement some of your experiments? Or best to wait and keep it simple?

Will porting the AppBar also be a step towards making it more composable, or is this just a 1:1 feature set port?

@mtrpcic A basic (and much more composable) AppBar is already in the next branch (https://github.com/callemall/material-ui/blob/next/src/AppBar/AppBar.js). It's fairly easy to implement the an AppBar with a similar API in older version of material-ui too.

@newoga any desire to implement some of your experiments? Or best to wait and keep it simple?

Sorry @nathanmarks! Didn't see that mention before. Yeah I'd be interested though as I have time probably makes sense to keep this component simple and work on porting other components.

@newoga Thanks for the info!

Im interested in the issue #5478 about having a Google Inbox style text field in the center of my app bar, what do the changes to AppBar mean for this?
I wouldnt mind using the Title property for doing it, but then if I want a title there, Id have to manually copy the h1 element with all its style properties which feels a little too dirty for my liking.

I'm very new to material-ui. I'm trying @next because I'd prefer start directly with the latest syntax and features that with 0.x.x even if there's no all features for now (I feel like I'm going to waste time for nothing by learning to use an old version).
<AppBar>Hello World</AppBar>
Gives

There's no height to the AppBar
My layout :
const palette = createPalette({
  primary: red,
  accent: pink,
});

const { styleManager, theme } = MuiThemeProvider.createDefaultContext({
  theme: createMuiTheme({ palette }),
});

function Layout(props) {
  return (
    <MuiThemeProvider theme={theme} styleManager={styleManager}>

          <Navbar />

            {props.children}

        {/**/}

    </MuiThemeProvider>
  );
}

@mbrookes @oliviertassinari Isn't it ported already and can be closed?

I think perhaps it was being kept open pending any refinements that @newoga had been working on.  I think we can close it for now.


Add API Support

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Add the ability to pull healthcheck directly from the API endpoint
Describe alternatives you've considered
None
Additional context
None

@robertlynch3 try
https://netbox.local/plugins/netbox_healthcheck_plugin/healthcheck/?format=json

I had the same request, but then I realized I can allow that endpoint by setting LOGIN_REQUIRED = False in configuration.py.  For me this is safe because I run behind a secure reverse proxy.


Plan workshop outline and topics

Definition of done

[x] Type the basic outline from brainstorming notes.
[ ] Add brief details about each topics into outline.
[ ] Determine ways to encourage and reward participation.
[ ] Figure out how to divide the audience into working groups.
[ ] Figure out how working groups will present their main findings.

Outline work in progress.


Customization

Hi Jeff
I'm using your cart on my site and I'm really impressed, I would like to customize it slightly and found some code on your FAQ page
My site is a Yola designed site and any CSS/HTML has to be inserted via a HTML widget, I tried your code after wrapping it in style tags like so (Obviously with the < >)
style type="text/css"
#myPage #PPMiniCart ul { height: auto; min-height: 130px; max-height: 500px; }
/style
but it didn't work. I would also like to change the checkout button image for one of my own but am unsure of the code needed.
Any help would be appreciated
Regards
David

Hi David,
I'm not familiar with Yola sites, but I can take a look at what's going on if you can send me a link.
Note, that the CSS example above requires that the html body tag have an ID of "myPage" on it (this can be changed to anything you'd like as long as it's reflected in the CSS rule. You may want to check that first.

Hello Jeff
Thanks for your very prompt reply, this is the page the cart is on
http://www.leathersolutions.org/leather-care-shop.php
Please accept my apologies for my lack of knowledge on this subject, until I built my site I had no idea what CSS and HTML was and have been slowly learning the basics. I'm assuming the style tags were the right thing to do.
My two hopes are to be able to apply the code above and to change the 'checkout' button image to match my custom 'add to basket' buttons.
After reading your last two line I feel sure this is the problem, unfortunately I have no idea what you're talking about so am unable to act on it.
Thank you for your help on this matter

Thanks for the link. It looks like the body ID of your page is actually "sys_background". The CSS from above should be this instead:
#sys_background #PPMiniCart ul { height: auto; min-height: 130px; max-height: 500px; }

If you want to override the checkout button background image you can do something like this in your CSS:
#sys_background #PPMiniCart p input { background: url(/path/to/your/background.png} 0 0 no-repeat; }

Hi Jeff
The auto sizing works perfectly, unfortunately the new checkout image doesn't, please could you take a look.

Sorry about that. It looks like there was a typo in my example (the url ended with a "}" rather than a ")").
I wouldn't normally go to this length, but I was playing around with it and I'm guessing you'll want something like this:
#sys_background #PPMiniCart p input {
     background: url(http://www.leathersolutions.org/resources/checkout.jpg) 0 0 no-repeat;
     text-indent: -1000em;
     width: 61px;
     height: 25px;
     border: 0;
}


Fix compatibility with recent rebar3

@Lucus16 wrote on Jul 11, 2019 8pm UTC:

Since nixpkgs switched to rebar 3.9, mixToNix is broken for rebar dependencies which have their own dependencies. Rebar deprecated hex registry v1 (https://github.com/hexpm/specifications/blob/master/registry-v1.md), so the fake hex registry no longer works, rebar doesn't even look at it anymore. It doesn't seem like they've switched to v2 either, which uses protobufs. They seem to be using a file called .cache/rebar3/hex/packages.idx instead, which is still in ETS format.
The nixpkgs commit where mixToNix broke is d4b243905f2181111b52d5a58b2119469ed689fc. It also removes a patch for rebar3 which seems to disable the update code. I tried applying it to rebar 3.11, but it still tries to talk to repo.hex.pm.
I think it's best to start using the custom fetchHex to place the sources in _checkouts since it was made for sandboxed builds. See https://www.rebar3.org/docs/dependencies#section-checkout-dependencies for more information. This only provides sources though, so I'm not sure independent compilation for each library is still possible. If the sources are provided in _checkouts, rebar might be convinced of using the artifacts in _build. If that's not possible, rebar dependencies may need to be built flat, without first building its recursive dependencies. I couldn't figure out how to add stuff to the _checkouts directory because the control flow of mixToNix is pretty messy.
I tested all this with a basic elixir project with a dependency on certifi and a recent nixpkgs unstable with 2c2fe05ed5a585194fdc98f71823d9937965a7ca reverted. Rebar dependencies that don't have dependencies themselves still seem to work.

@yorickvp wrote on Jul 11, 2019 8:57pm UTC:

PR for build-mix fixing: https://github.com/NixOS/nixpkgs/pull/64884

Status update: I couldn't make _checkouts work without reducing functionality, so I've been working on a rebar3 package index generator:
defmodule FakeRebar3PackageIndex do
  @package_index :package_index_v4
  @package_index_version 4

  def dependency(name, version) do
    [package: name, requirement: "~> #{version}"]
  end

  def package(p) do
    [key: {p["name"], p["version"], ""},
     checksum: p["sha256"],
     retired: false,
     dependencies: (for d <- p["deps"], do: dependency(d["name"], d["version"]))]
  end

  def main([out]) do
    packages = Jason.decode!(IO.read(:all))
    table = :ets.new(@package_index, [:named_table, :public, :ordered_set, {:keypos, 2}])
    :ets.insert(table, {@package_index_version, :package_index_version})

    for p <- packages, do: :ets.insert(table, package(p))
    :ok = :ets.tab2file(table, String.to_charlist(out))
  end
end

This doesn't work just yet, I think I'm getting there.


Feature :: Field Note Option

Unless I somehow overlooked it, there is no way to currently add a "note" to a field. So, would there be interest in adding a new option to all fields called Note? There are times where a more detailed description of expected input would be helpful to the user than just a field label by itself. Something like:
Twitter Handle
@gilgnyc for example.
Probably a terrible example but hopefully you get the point. I can try to mock up a UI example tonight.
I mainly want to see if this is something that would be considered for core before I spent a lot of time adding it to the code base.

I think this is a great idea, I'm currently creating a website powered by Netlify CMS which has multiple contributors and it would be great to add additional descriptions to fields to help editors better understand the types of content they should use.

This seems like a duplicate of https://github.com/netlify/netlify-cms/issues/552 to me -- if so can we move the discussion there?

Indeed it's along the same lines. Sorry about this. Searched for 'Note' but didn't find anything so I opened this.
Will close.


What does "dual licensing" related to JSqlParser in practice mean?

I know there were similar discussions here about this, but I read them and I didn't find an answer.
My company develops and sells a software product, and JSqlParser does a great job in one area. Now we got a new customer and they tell us that they accept 3rd party libraries in our product only when they are licensed under Apache V2, but not when they are licensed under LGPL. I'm not a lawyer (as probably most readers here), so I'm a bit confused what "dual licensing" actually means.
Best case: Does it mean that I can apply one of the two licenses, i.e. the one I prefer? see http://www.dpscomputing.com/blog/2012/06/27/what-dual-licensing-means-in-terms-of-computer-software/
Worst case: Or does it mean that both licenses apply at the same time, i.e. one could pick the most restrictive points from each? see https://en.wikipedia.org/wiki/Multi-licensing
Which case applies?
Thanks in advance.

In the last days the licensing was a field of change.

The dual licence text was changed to reflect, that there is a "or" betewen both license and not an "and"
The header of each sourcecode file does now not anymore contain the LGPL license but a text refereing to both licenses

sounds great, thanks a lot!


Failed to format TF file when the key value of tag is a number

Hi,I had a problem with terraformer. When the key value of the resource tag is a number, formatting TF file fails, which results in the template file not generating normally. Such as :
resource "****" "***"{   tags = {     123 = "123"     qwe = "qwe"   } }
Error message：
expected: IDENT | STRING | ASSIGN | LBRACE got NUMBER

Can you add example?
expected and actual? which provider and resource?

@sergeylanzman Any resource that starts with a number or has a bool value will report an error. Alicloud provider,image resource. You'd better try to create a resource whose key value is a number and export it. For example, the tag key value is 123 or true or false


[Cherry-pick] Fix bad column name in mesh dashboard for TCP

Moving #9337 to master.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
If they are not already assigned, you can assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@douglas-reid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
d77d4d2f3ca6c7043c10d6d2b237fb9360d03201
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hklai
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [hklai]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Write a GO program to find average of digits of a number by recursion

Description
Write a GO program to find average of digits of a number by recursion
Average of digits of a number is the sum of digits of the number divided by the number of digits in the number. For example, the average of digits of 123 is 6. The sum of digits of 123 is 6 and the number of digits in 123 is 3. So, the average of digits of 123 is 6/3 = 2.
Input  : 123
Output : 2

How to contribute

Save the solution in find_average_of_digits_of_a_number_by_recursion.go file
Add find_average_of_digits_of_a_number_by_recursion.go file in program/find-average-of-digits-of-a-number-by-recursion folder

!assign


Add guide: Configure clients for Observatorium

Ping (:

this sounds like we should probably note that. Maybe it means we should keep this section out of this doc for now and put it into a new PR that we merge once the PO and CMO add this functionality. Otherwise I worry this will create more confusion. wdyt?

I have removed the draft content for now, and I am maintaining a local draft. Whenever we get the OAuth2 functionality in OpenShift, we can update the doc with proper OpenShift versions.


CSRF protection return 403 because of removing other cookies

there is django csrf protection
on calling filedownload post data that return csrf failed
https://docs.djangoproject.com/en/1.9/ref/csrf/

i dont use django but certainly let me know if you find a solution and/or make a PR

For Django CSRF token protection
fileDownload need add csrftoken parameter
currently, you just need add correct csrf value to do it
Example:
$.fileDownload(url, {
    httpMethod: "POST",
    csrfmiddlewaretoken: "{{csrf_token}}",
    parameter: your_paramter,
})

There's a problem with iPad and django (1.11.6) on POST requests.
It seems Safari is not setting/sending the document.referrer, so the CsrfViewMiddleware will reject the request with error 403 because it suspects about a man-in-the-middle attack.
fileDownload when dealing with isIos devices opens a new window.


Waiting for failover to finish

I'm testing out the failover client and wondering what the best approach is to keeping my subscriber process waiting while it attempts to reconnect.
Right now I have a simple loop in my main process
while client.connected?
  sleep 1
end

During a failover (i.e., if I shut down the broker it was talking to), this loop breaks because client.connected? returns false.
What event(s) should I use to determine whether or not the client is going to be attempting a retry?

I would rely on a separate sentinel to keep track of when to terminate your main loop. For example, if you want to send 5 messages and quit only when you've ACK'd receipts for the 5, then you could do something like:
sent_messages = 0
client.on_receipt { |r, *_| sent_messages += 1 }

while sent_messages < 5
  sleep 1
end

Be warned, depending on how you're approaching failover/retrying, you can run into some issues as outlined in #13 and the failover functionality found in onstomp/failover can suffer from race conditions related to this issue.

Thanks. I'm glad you've at least put some initial thought into these race conditions.
My intent is to have a long-running queue subscriber which simply does its best to stay connected and wait around for new messages and be allowed to failover to our standby broker when the master goes offline.
Looks like my best bet right now is to wait for failover_retries_exceeded to terminate the main loop if the client URI is failover. If we're not in failover mode, then continue to rely on .connected?


Add an optional only_if_main arg to allow placing flags.mark_flag[s]_as_required next to the flag definitions

There's a readability benefit in placing flag validations together with the flag definitions. However, flag definitions are customarily placed near the top of the file, while the main block (if __name__ == '__main__':) is customarily placed at the bottom of the file. If we add an only_if_main arg that allows suppressing the behavior when the module is imported, users will be able to place calls to flags.mark_flag_as_required and flags.mark_flags_as_required wherever they wish.

See https://github.com/abseil/abseil-py/pull/150.


 I'm planning to use MJML in a commercial context where the email design will be done using MJML and then the generated HTML will be sent to customers.
I've checked the generated HTML (using MJML VS Code extension) and can't find MIT license being parsed. Would it cause me trouble if the license isn't there on the generated HTML?

There's no need to have the MIT license in the final output, it's fine for you 👍


Reponsiveness of Category Page

Is your feature request related to a problem? Please describe.
My suggestion is to make category page as responsive it is not at all responsive in screen size lesser than 1024px and also hide the navbar option in smaller size screen so that it would be pleasing to eyes
Describe the solution you'd like
I thought of hiding the navbar option in smaller size screen and will add the hamsburger in order to show option and i will also align all the items to it correct position
Additional context
Add any other context or screenshots about the feature request here.

someone is working on the mobile responsiveness for whole website !


Registering systems via world.register_system are unable to be excluded from despawn recursive patterns

Bevy version
0.13.1
What you did
Using a standard cleanup pattern between app states removes any dynamic systems registered with world.register_system(...)
pub fn cleanup_entities(
    mut commands: Commands,
    query: Query<(Entity, Option<&Name>), (Without<Persistent>, Without<Parent>, Without<Window>)>,
) {
    for (entity, name) in query.iter() {
        info!("Removing entity with name:{:?}, id: {:?}", name, commands.entity(entity).id());
        commands.entity(entity).despawn_recursive();
    }
}

What went wrong
The system is registered using
pub fn register_boxed_system<I: 'static, O: 'static>(
        &mut self,
        system: BoxedSystem<I, O>,
    ) -> SystemId<I, O> {
        SystemId(
            self.spawn(RegisteredSystem {
                initialized: false,
                system,
            })
            .id(),
            std::marker::PhantomData,
        )
    }

This inserts a RegisteredSystem
However this struct is not public so I am unable to exlude it from my cleanup.
Additional information

This is effectively a duplicate of #12852: I'm going to close this and encourage you to participate there :)


Upgrade Go to 1.22.2 to get rid of CVE-2023-45288

Upgrade to the latest version of Go to get rid of CVE-2023-45288 in our docker images.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: nacx
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [nacx]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@f476d17). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #244   +/-   ##
=======================================
  Coverage        ?   90.31%
=======================================
  Files           ?       24
  Lines           ?     1569
  Branches        ?        0
=======================================
  Hits            ?     1417
  Misses          ?      108
  Partials        ?       44

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: nacx, sergicastro
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [nacx]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Config Files Differ

I'm new to PMB, but got it initially working last night.  I started over as for some reason I can no longer get TuneIn to show any stations.  So, on a fresh install, I booted first time with no edits to settings.ini.  After boot, editted setting via web interface to add ssh and turn on a shairplay-sync and upnp.  Mopidy server shows up fine on MPod, but will hang on getting songs and not connect.  I check the settings.ini and mopidy.conf and they are identical for both http and mpd.  host is <IP_ADDRESS> and zeroconf = "" (default settings).  When I execute mopidy config, the http and mpd sections have host = <IP_ADDRESS>. and several other settings in the mpd section that are not in either settings.ini or mopidy.conf.  I know that using loopback host is causing MPod not to connect, but where would I find the config file that is currently being used my mopidy and change it to be inline with settings.ini/mopdiy.conf.

Found the default config in /root/.config/mopidy/mopidy.conf.  I am assuming that if you want to change any of the values (uncomment them) and make a value change (i.e. host = <IP_ADDRESS>) that these become the new defaults.  Shouldn't changes to values in settings.ini be applied after the default values if the keys are the same?

When mopidy runs as a system service, as it is designed to do in pimusicbox, it uses the config file in /etc/mopidy/mopidy.conf. As it happens, in pimusicbox this file is actually a symlink to /boot/config/settings.ini. That root config file is used when you run mopidy as the root user I.e. not a system service. The default are actually elsewhere and really don't need to be touched ever. I have not tried mpod in a long time but I do remember it being a poor experience.

Thanks for the quick response.  If mopidy is running as a service, is there a special way to check it's loaded config that is different from checking mopidy config?  I can't run mopidyctl config as mopidyctl is not found.  It seems that changes made to /etc/mopidy/mopidy.conf are not being made when the service is restarted.  I can tell because the zeroconf name is staying at the default even if I change it in settings.ini or conf file directly.

You don't really want to be changing zeroconf in there, I think the comment should specify that it's set like that to disable it. That's because we use the system zeroconf settings for everything rather than having various services implement it themselves. I'd have to check where that zeroconf file lives I don't remember off the top of my head.
I also don't recall what the appropriate alternative to mopidyctl is in pimusicbox v0.7. I think it might be as simple as 'service mopidy run config'.

"service mopidy run config" works.  I'm not worried about the name, but I was changing it because it shows in MPoD even if I don't connect to it during this testing.  If I change the root's hidden config host info to <IP_ADDRESS>, the service config changes accordingly and connection to MPoD works fine and operation is normal.  Now I just have to figure out why I get no TuneIn stations being displayed.  Last night I was getting everything listed just fine.  Any ideas on troubleshooting TuneIn?  In the service config it is enabled and timeout specified.  All I get is Trending with nothing in it when I browse there in MPoD or web client.

Came home 3 hrs later tonight and TuneIn is showing everything fine.  Not sure what the issue was, but it is working fine.  Thanks for the info on configs - 2 more Pi's ordered to replace my old asus routers running mpd.

There are still some settings that are not mapping through settings.ini to the running mopidy config.  In settings.ini I have turned off everything except tunein, but after save and reboot, when you hit browse in the webclient,  files, spotify browse, tunein, itunes-podcasts are still there.  When I run service mopidy run config, it shows podcast-itunes true (podcast shows false), spotify_tunigo true (spotify and spotify_web show false), and file true (local shows false)  It looks like there is a conflict in labels between settings.ini and mopify.conf.  I hope the way I'm explaining this makes sense.  I would try to disable these manually, but I don't want to mess anything up.

You can close this - I know what is going on, but use it for your info if you want settings.ini to configure all the options in the default config and clean up things so the ui can control all the items displayed in browse.  Thanks for the quick responses.


--profiles fails using Command Line

Trying to make a query like this one:
twitterscraper gundam -l 10 -bd 2019-8-22 -ed 2019-8-23 -o gundamFile.json --profiles
Creates the JSON file but fails with the profiles:

Traceback (most recent call last):
File "/anaconda3/bin/twitterscraper", line 10, in
sys.exit(main())
File "/anaconda3/lib/python3.7/site-packages/twitterscraper/main.py", line 126, in main
list_users = list(set([tweet.user for tweet in tweets]))
File "/anaconda3/lib/python3.7/site-packages/twitterscraper/main.py", line 126, in
list_users = list(set([tweet.user for tweet in tweets]))
AttributeError: 'Tweet' object has no attribute 'user'

I have the same problem, need help

@johannstark @Lidl-to-go  Check this Improvement fork i made to a twitter scraping package originally built by Jefferson Henrique.