Kamel CLI run from URL

You can provide a file as an argument to run an integration, but we also need to be able to run an integration from a github/URL repository. Like:
kamel run -d camel-http https://raw.githubusercontent.com/hguerrero/camel-netty-proxy/master/src/main/java/com/github/zregvart/cnp/ProxyRoute.java
Kamel CLI should pass the URL directly to the operator or can download locally the file and then send it to the operator.

this should be supported, does ti fails ?

This is the error when trying:
integration "proxy" created
Error: stat /Users/hguerrer/https:/raw.githubusercontent.com/hguerrero/camel-netty-proxy/master/src/main/java/com/github/zregvart/cnp/ProxyRoute.java: no such file or directory
Usage:
  kamel run [file to run] [flags]

Flags:
      --compression             Enable store source as a compressed binary blob
      --configmap strings       Add a ConfigMap
  -d, --dependency strings      The integration dependency
      --dev                     Enable Dev mode (equivalent to "-w --logs --sync")
  -e, --env strings             Set an environment variable in the integration container. E.g "-e MY_VAR=my-value"
  -h, --help                    help for run
  -k, --kit string              The kit used to run the integration
      --logging-level strings   Configure the logging level. E.g. "--logging-level org.apache.camel=DEBUG"
      --logs                    Print integration logs
      --name string             The integration name
      --open-api strings        Add an OpenAPI v2 spec
  -o, --output string           Output format. One of: json|yaml
      --profile string          Trait profile used for deployment
  -p, --property stringArray    Add a camel property
      --repository strings      Add a maven repository
      --resource strings        Add a resource
      --secret strings          Add a Secret
      --sync                    Synchronize the local source file with the cluster, republishing at each change
  -t, --trait strings           Configure a trait. E.g. "-t service.enabled=false"
  -v, --volume strings          Mount a volume into the integration container. E.g "-v pvcname:/container/path"
  -w, --wait                    Waits for the integration to be running

Global Flags:
      --config string      Path to the config file to use for CLI requests
  -n, --namespace string   Namespace to use for all operations

Error: stat /Users/hguerrer/https:/raw.githubusercontent.com/hguerrero/camel-netty-proxy/master/src/main/java/com/github/zregvart/cnp/ProxyRoute.java: no such file or directory```

It works normally.. Except for the "package" issue #1132.
That error happens only if the user enables dev mode.. watching the remote http file does not make a lot of sense, maybe we can just warn..


Add Squadron allegiance icon

Squadrons have an allegiance icon

svg: https://drive.google.com/open?id=1fdJhoCz2Ugl__Tu4ETsNOr58l4D_N6Tk


Provide ability to extend or overwrite cloud resource tags

Related to the following PR: https://github.com/labd/mach-composer/pull/57
MACH sets resource tags by default (at least; in Azure for now).
It should be possible to either extend or overwrite them in MACH with an extra tags setting in your Site configuration.
I think being able to overwrite them gives you the most flexibility. I think we should not provide both options (overwrite and extend) because it will make the configuration more complex.

maybe we can implement this in away that feels the same as AWS: #143

I think this first requires https://github.com/mach-composer/mach-composer-cli/issues/244, so we can change the plugin protocol and allow for setting and handling tags


Update to Python 3

Update the code for python 3 use (refactoring and requirements upgrade)

No issues page so I'm posting here
I am trying to use the pyfixit library, and the documentation is set up well https://pyfixit.readthedocs.io/en/latest/index.html, but it doesn't explain what the output or result of the program is
Does pyfixit update a guide to the Dozuki website?
Does it only create local dozuki guides?
How would I connect the work I write in pyfixit to my Dozuki account?
Thanks


Hotfix for "Hiding columns in Table does not work #374"

Hotfix for the problem related on #374 about the issue on Hide/Show columns when user clicks on "Hide" or "Show columns" options

Replaced by https://github.com/neo4j-labs/neodash/pull/685


Pager Component Accessibility

Adding Labels and Roles for Accessibility
What kind of change does this PR introduce? (check one with "x")

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
Pagination options are not labelled and not marked up as buttons
Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[X] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

@amcdnl
Can you please take a look at this PR? Thanks!

LGTM

@amcdnl Do you know when will this patch be included in a new release? Thank you.


Adding a static route breaks graphql route

api = responder.API()
api.add_route("/graphql", schema)

If I add a route for my graphql server, I can post to it and get an api response (from graphiql for example)
api = responder.API()
api.add_route("/graphql", schema)
api.add_route("/", static=True)

If I add a static route (or any route with default=True, I now get the index.html file back. The graphiql interface still loads but when I make a query I get index.html back as the response.

Working fine for me

I'm using a custom static dir
api = responder.API(static_dir='build')

Perhaps this doesn't do what I thought it did. I thought it would mean it just used the folder build instead of the folder static to serve static assets.
The problem goes away if I remove static_dir

It's doing what it was build to do.

While debugging I've found this bug
graphiql template is loaded into resp.content, so it's served first and the static response (loaded into resp.text) is shadowed.

This makes sense - if the index.html is missing nothing is written into resp.text and the response is still valid json.

Once graphiql is loaded, it makes some requests, that's when resp.text is sent

Because we sent first resp.content if there's any else resp.text if it exists than the formats and the media

Just made a PR #142 please check if it resolves the problem

Nope, but if you want that behavior don't set static to True

From the docs:
default – If True, all unknown requests will route to this view.
static – If True, and no endpoint was passed, render “static/index.html”, and it will become a default route.
Static is the default route, and default routes are supposed to match unknown requests. If I've already registered the route, then it's not an unknown request.

My bad, I get you know !

Hey @davidblurton, test PR #147

@taoufik07 Tested it out - this is perfect 🤩

@kennethreitz guess it's fixed, you can now merge #147 and close this issue :D


Update travis script "pretty-check" to add more config features

This commit adds the following to the configure options under
"pretty-check" build in travis script:

enable-ncp,
enable-jam-detection,
enable-raw-link-api.

This ensures that source files for these features are included in
the travis "pretty-check" build.

Codecov Report

Merging #1569 into master will increase coverage by 1.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1569      +/-   ##
==========================================
+ Coverage   69.63%   70.76%   +1.13%
==========================================
  Files         157      156       -1
  Lines       20130    19192     -938
  Branches     2483     2390      -93
==========================================
- Hits        14018    13582     -436
+ Misses       5216     4742     -474
+ Partials      896      868      -28

Impacted Files
Coverage Δ

examples/platforms/posix/misc.c
0% <0%> (-66.67%)
:arrow_down:

src/core/thread/lowpan.hpp
86.84% <0%> (-13.16%)
:arrow_down:

src/ncp/ncp_base.cpp
26.7% <0%> (-7.93%)
:arrow_down:

src/core/net/ip6.cpp
53.91% <0%> (-3.81%)
:arrow_down:

src/core/net/netif.cpp
56.97% <0%> (-3.49%)
:arrow_down:

examples/platforms/posix/uart-posix.c
59.55% <0%> (-3.38%)
:arrow_down:

src/core/thread/lowpan.cpp
92.47% <0%> (-1.44%)
:arrow_down:

src/core/common/message.cpp
94.47% <0%> (-1.39%)
:arrow_down:

src/core/coap/coap_client.cpp
67.95% <0%> (-1.11%)
:arrow_down:

src/core/thread/mle.cpp
81.7% <0%> (-0.58%)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 009dd36...3f3f8da. Read the comment docs.

I believe the changes in #1620 makes this PR unnecessary.

Yes. Exactly.


withFileFromClassPath fails when path to classpath resource contains spaces

Hello,
How to replicate:

create directory with spaces in path

mkdir ~/dir\ with\ spaces

move some project using testcontainers-java to that directory
put a file to copy into container in resources directory (touch src/test/resources/any_file in this example)
use container loading files from class path, ie:

new GenericContainer(
        new ImageFromDockerfile()
                .withDockerfileFromBuilder(builder -> {
                        builder
                                .from("alpine:3.2")
                                .copy("any_file", "any_file")
                                .build();
                        })
                )
                .withFileFromClasspath("any_file", "any_file")
)

This will throw exception:
1 error
Caused by: org.testcontainers.containers.ContainerFetchException: Can't get Docker image name from org.testcontainers.images.builder.ImageFromDockerfile@61ae0d43
Caused by: java.lang.RuntimeException: Can't get size from ~/dir%20with%20spaces/your-project/target/test-classes/any_file
Caused by: java.nio.file.NoSuchFileException: ~/dir%20with%20spaces/your-project/target/test-classes/any_file

Issue is related to urlencoded spaces in path (%20) when using
URL resource = ClasspathTrait.class.getClassLoader().getResource(resourcePath);
String resourceFilePath = new File(resource.getFile()).getAbsolutePath();

in default implementation of withFileFromClasspath. Calling .toURI() instead of .getFile() on resource should fix this issue while not breaking anything - I'm using this workaround and seems to be fine. No PR though, sorry, simply no time to do it.

Hi @lbialy
Thanks for reporting, and sorry for the problem and thorough analysis. We'll try and address soon.
Richard

@rnorth I think you wanted to mention #262, not #263 here. Glad to see something found.

@lbialy this is hopefully fixed now - please could you let me know if you encounter any more issues?
I've been developing under a path containing space for a while, just to make sure 😃

Hi @lbialy
We released 1.3.0 with a few fixes to file mappings, could you please try it?
Thanks!

I'll try, thanks!

Fixed by #279.


WinForm app and Menu

Hi,
I have a simple windows app - windows forms, .Net 4 in c#
I'm trying to implement clicking on a menu and once i expand/click the first menu item, I can not click on the second one ..
Here is the code
windows.FindFirstByXPath("/MenuBar/MenuItem[@Name='File']").AsMenuItem().Expand(); // menu is succesfuly expanded
windows.FindFirstByXPath("/MenuBar/MenuItem/MenuItem[@Name='Add']").AsMenuItem().Click(); // no cigar

I do have some logic which should help when using menues. Can you try using the Menu class? See the example here: https://github.com/Roemer/FlaUI/wiki/Menu
Otherwise could you provide a sample application?


Remark cli

These commits have been incorporated: https://github.com/econ-ark/REMARK/commit/717319bb8d2c03fdf1befb9815755f62e9a132d0


Is it possible to get the text in a element line by line

Let's say there's a html like this,

      Line 1

      Line 2

I managed to get the text inside the
 tags. Is it possible to get each lines separately line by line ?

Hello,
There is no way to do this directly, but you can take a look at my comment on another similar issue, it should work in your case too: https://github.com/PuerkitoBio/goquery/issues/287#issuecomment-471043149 (i.e. get the contents of the  and iterate over the nodes, finding the #text nodes).
Hope this helps,
Martin


added customer churn to techzone

Adding Customer Churn Prediction to test the model deployment speed on OCP

Tested on sandbox-ocp. Closing this PR


"you should uncheck the implicit grant settings in the app registration" - what are those "implicit grant settings"?

... and why (and how) should we uncheck them?
Is it about disabling ID tokens? But according to other docs and our real-life experience, ID tokens are still required even for auth code + PKCE...
Can you please clarify?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d2e35563-a9af-c235-f151-7771cee56326
Version Independent ID: f196053d-a74e-b19f-bb7d-8470d73942dc
Content: Migrate JavaScript single-page app from implicit grant to authorization code flow - Microsoft identity platform
Content Source: articles/active-directory/develop/migrate-spa-implicit-to-auth-code.md
Service: active-directory
Sub-service: develop
GitHub Login: @mmacy
Microsoft Alias: marsma

@scale-tone
Thanks for your feedback! We will investigate and update as appropriate.

They are these settings that are under the Authentication menu in the app registration:

Select the tokens you would like to be issued by the authorization endpoint:
Access tokens (used for implicit flows)
ID tokens (used for implicit and hybrid flows)

You need the ID token if you are using hybrid flow, since it mixes implicit grant with the authorization code. In this case the application requests an ID token when it requests the authorization code. Authorization Code Flow by itself uses
code instead of id token. https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-oauth2-auth-code-flow#request-an-id-token-as-well-hybrid-flow
This is documented pretty well in this OIDC guide: https://www.ibm.com/docs/en/sva/9.0.5?topic=SSPREK_9.0.5/com.ibm.isam.doc/config/concept/con_oauth20_workflow.html
I made a pull request to add the location. https://github.com/MicrosoftDocs/azure-docs-pr/pull/185138
The section under the app registration also mentions, "If the application has a single-page architecture (SPA) and doesn't use the authorization code flow, or if it invokes a web API via JavaScript, select both access tokens and ID tokens. For ASP.NET Core web apps and other web apps that use hybrid authentication, select only ID tokens."
Let me know if this helps at all. @mmacy feel free to add if I missed anything.

Thanks @MarileeTurscak-MSFT , but just to clarify and confirm again:
This page under question is all about JavaScript Single-Page Apps and the new auth code+PKCE flow (correct?).
The page instructs us to use the new @azure/msal-browser library.
That library (according to our practical experience) does uses that "hybrid flow" you mentioned. At least, as of today.
What that all means, that the correct combination of settings for this scenario should be:

Access Tokens - disabled
ID tokens - enabled.

Did I get you correctly?
If yes, then I'd suggest to explain it exactly like that on this page, to avoid misunderstanding.

@scale-tone I agree this was unclear. If you try to disable ID tokens for your application you are gonna get a warning that you are about to disable implicit grant settings, which would imply that ID tokens should in fact be disabled in this case.
According to this documentation, none of the checkboxes should be selected https://learn.microsoft.com/en-us/azure/active-directory/develop/scenario-spa-app-registration#redirect-uri-msaljs-20-with-auth-code-flow (step 4)
I agree that some official clarification on this would be nice since the migration guide is not clear on this.


Vulnerabilities detected on jenkins image

Hi all,
these vulnerabilities were detected on latest jenkins image present on docker hub: ( 2.60.3-alpine)
CVE-2017-8816
CVE-2017-8818
Critical
curl
7.56.1-r0                                 fixed version  7.57.0-r0
CVE-2018-0500
Critical
curl
7.56.1-r0                                 fixed version 7.61.0-r0
CVE-2018-1000005
CVE-2018-1000007
Critical
curl
7.56.1-r0                                 fixed version 7.58.0-r0
CVE-2018-1000120
CVE-2018-1000122
Critical
curl
7.56.1-r0                                 fixed version 7.59.0-r0
CVE-2018-1000300
CVE-2018-1000301
Critical
curl
7.56.1-r0                                fixed version 7.60.0-r0
CVE-2018-14618
CVE-2018-16839
CVE-2018-16840
CVE-2018-16842
Critical
curl
7.56.1-r0                             fixed version 7.61.1-r1
CVE-2018-17456
Critical
git
2.11.3-r0                             fixed version 2.11.3-r2
CVE-2018-14599
CVE-2018-14600
Critical
libx11
1.6.4-r0                              fixed version 1.6.6-r0
CVE-2017-5461
Critical
nss
3.27.2-r1                          fixed version 3.28.4-r0
CVE-2018-2938
Critical
openjdk8
8.121.13-r0                      fixed version 8.181.13-r0
CVE-2018-3183
Critical
openjdk8
8.121.13-r0                     fixed version  8.191.12-r0
CVE-2017-10989
Critical
sqlite
3.15.2-r0                         fixed version 3.15.2-r1
CVE-2016-9841
Critical
zlib 1.2.8-r2                    fixed version 1.2.11-r0
CVE-2016-9843
Critical
zlib
1.2.8-r2                          fixed version 1.2.11-r0
We are trying to install a custom repository with some interesting features for our consumers.
As a registry we installed Harbor with "Trivy" add-on as vulnerability scanner.
When I pushed the latest image of jenkins from docker hub to our registry "Trivy" detected the vulnerabilities above marked as "Critical".
Could you let us know when you fixed the image with the correct libs and packages version or if we could use another image?
Thanks in advance.
Regards,
Gabriele

@gabrilabs75 that image has been deprecated for over 2 years, see warning: https://hub.docker.com/_/jenkins
Please see the tags for the jenkins/jenkins image:
https://hub.docker.com/r/jenkins/jenkins
We're mostly dependent on upstream maintainers for minimal CVEs,
alpine, slim and centos are your best bet I believe.

@timja thanks Tim! I didn't notice that...
Anyway I tried to push this version 2.263.1-lts-alpine and Trivy found these vulnerabilities marked as "High"
CVE-2020-8231
High
curl
7.69.1-r1               fixed on 7.69.1-r2
CVE-2020-8285
CVE-2020-8286
High
curl
7.69.1-r1               fixed on 7.69.1-r3
Could it depend on the "starting" operating system ?

Note that for system components, we just deliver what upstream provides at the time the image is created. While there may be some choices we can make about the default and available variants (e.g. https://github.com/jenkinsci/docker/issues/1033), upstream is generally far better equipped to address system-level issues than the Jenkins project.
In this case, Alpine addressed those CVEs in 3.12.3: https://www.alpinelinux.org/posts/Alpine-3.12.3-released.html
Which, according to https://alpinelinux.org/downloads/, was

Released Dec 16, 2020

while 2.263.1 was released Dec 3. LTS releases are on a roughly monthly release cadence, if you need more than that, you will need to do it yourself.

Alpine 3.12.3 is included in the Jenkins 2.263.3 Docker image as reported by:
mwaite@mark-pc2:~$ docker run --rm -i -t jenkins/jenkins:2.263.3-alpine cat /etc/os-release
Digest: sha256:fa0b0f06383944877c8105ad8e7edcac29f05583cef8b0d26cd8c9586e0cb667
Status: Downloaded newer image for jenkins/jenkins:2.263.3-alpine
NAME="Alpine Linux"
ID=alpine
VERSION_ID=3.12.3
PRETTY_NAME="Alpine Linux v3.12"
HOME_URL="https://alpinelinux.org/"
BUG_REPORT_URL="https://bugs.alpinelinux.org/"

Alpine 3.12.3 is included in the Jenkins 2.263.3 Docker image as reported by:
mwaite@mark-pc2:~$ docker run --rm -i -t jenkins/jenkins:2.263.3-alpine cat /etc/os-release
Digest: sha256:fa0b0f06383944877c8105ad8e7edcac29f05583cef8b0d26cd8c9586e0cb667
Status: Downloaded newer image for jenkins/jenkins:2.263.3-alpine
NAME="Alpine Linux"
ID=alpine
VERSION_ID=3.12.3
PRETTY_NAME="Alpine Linux v3.12"
HOME_URL="https://alpinelinux.org/"
BUG_REPORT_URL="https://bugs.alpinelinux.org/"


Improved Plugin installation UX - phase 1

What is the areas you would like to add the new feature to?
Notation CLI
Is your feature request related to a problem?
For remote signing, users need to install a signing plugin into notation. Here is the current UX of plugin installation:

Download the plugin from the plugin vender repository
Figure out what is notation configuration directory based on the OS type
Make a directory named plugins under notation configuration directory
Make a sub-directory under plugins for the vendor plugin (of cause, users can use command mkdir -p)
Extract the plugin package (if needed) and copy the plugin file into directory created in step 4
Use command notation plugin ls to make sure the plugin is installed

It takes so many steps to just install a plugin, and it is also error prone since most steps are manual steps and user need to figure out the directory path accordingly.
What solution do you propose?
To improve the plugin installation UX and meet the timeline of v1.0 release. A phased approach is recommended. This issue only addressed the solution for the first phase.
The goal of the first phase is that users don't need to create directories, remember the names, and verify the installation using notation plugin ls. The solution is to introduce two new sub-commands for notation plugin:

notation plugin --name <plugin name> import <plugin file>
notation plugin delete <plugin name>

With that, the steps of plugin installation can be reduced to two steps

Users download the plugin from the plugin vendor repository.
Users execute command notation plugin import <plugin package>. The result of the execution indicates the success or failure of installation.

Users can easily delete plugin as well using just one command: notation plugin delete <plugin name>. If users forget the plugin name when deleting, users can use the existing command notation plugin ls to check the name.
What alternatives have you considered?
None
Any additional context?
No response

Update to RC-3 milestone per community discussion on Configuration UX improvement

@yizha1 is there anyone actively working on this feature? If not, I'd like to start working on it. My team and I ran into this exact scenario where each of us were on a different OS and had to figure out how to install the plugins. Imo, adding this feature would greatly improve the user experience and I'd like to help. :)

@Duffney Great and your contributions are welcome! 👍
Currently it is not assigned to anyone yet. You can take it. Here are some steps we are now practicing on a feature development:

Share the design/solution of the feature in the community.
PRs to update the specification if it is necessary, for CLI command, we should update the specs under the directory. For this feature, it is plugin.md. I can work with you on the spec if you need any support on it.
PRs for implementation

This feature is planned for rc.3, and the current target date is around the first week Apr. Let me know if you have further questions.

Awesome! I appreciate you sharing the timeline with me also. My next question is, what do you mean by "in the community"? Are you referring to posting the solution in the Slack community or as agenda item for a community call? Either way I'll get started with that and tag you in the PR for the plugin spec changes.
Thank you so much @yizha1 ! :)

@Duffney No problem.
WRT to the Share the design/solution of the feature in the community, you can choose the best way to do it as long as it is open discussed and tracked in GitHub. I saw some practices today. In some cases, the solution was discussed in the Github issue directly maybe due to size of feature or solution is small. In some cases, folks use hackmd.io to share and present the solution in the community call, especially for big size of solution or feature. It is a good practice to request an offline review first before presenting in the community call, and then, give a brief intro and address critical comments during community call. In this way we can improve the efficiency of review and save time for other people. Hope it helps, and you can decide the best way to do it.

That makes perfect sense, thank you for the explanation. Here's what I was thinking based on the information in the issue:
Solution: Improve the plugin installation user experience by adding two subcommands to the notation plugin command that install and delete plugins for the user when provided with the plugin package and or name.

notation plugin import will create the necessary directories, remember plugin names, and verify the installation for notation plugins.
notation plugin delete will remove a specific plugin from notation given the plugin name and delete the plugin package files.

Please let me know if you think this size of issue warrants a hackmd.io post and community presentation.

Here's a link to the spec PR. If it's easier to collaborate outside the PR, please lmk. :)

@priteshbandi @vaninrao10 @shizhMSFT @FeynmanZhou PTAL at Josh's proposal for this issue starting from this comment https://github.com/notaryproject/notation/issues/549#issuecomment-1431822034. If needed, we can invite Josh to the community meeting for further discussion.

Move this feature to post v1 release.

Been having a chat with @toddysm about the challenges of installation right and it seems to be an challenging problem.
There are some opportunities to streamline plugin installation and I would like to recommend we provide a simpler experience.
For e.g.
Can we enable a brew install like semantics for plugins?
notation plugin install ghcr.io/wabbit-networks/notation-plugin@sha256:....
The goal of this to ensure that plugin installation is taken care of by an OCI artifact that will include the binary and also download and install the binary in the appropriate config location. Second it is digest verified and the binary executable blob can be verfiied against the manifest hash. Lastly if there is a signature attached with the plugin itself, notation can verify it at and we can implement this at a later phase.

@sajayantony If the plugin file is packaged into an OCI artifact, then it can be downloaded and installed from an OCI registry. Plugin developers can sign and add SBOM to the OCI artifact type of plugin to ensure security and integrity in its distribution. I like this idea.
@notaryproject/notaryproject-notation-maintainers  Could you take a look at this proposal? If we have an agreement on this proposal, I think the next step would be to update the Spec plugin-extensibility.md and related CLI spec to define the overall plugin conventions and CLI experience.

LGTM
@Duffney Please take a look at this proposal  by @sajayantony . Since this issue is out of v1 scope now, we have more time, maybe we can aim at the final experience.
@FeynmanZhou I agree with your last comment, besides that, we also need to add user document for plugin on website.

Sharing an implementation by @noelbundick-msft  https://github.com/deislabs/ratify/pull/519

notation plugin install should also support at least one local resource in case of airgapped networks.
notation plugin install <registry>/<repository>@<digest>
notation plugin install --file <tarfile_path> --checksum <digest>

notation plugin install should also support at least one local resource in case of airgapped networks.
notation plugin install <registry>/<repository>@<digest>          # remote source
notation plugin install --file <tarfile_path> --checksum <digest> # local source

This is a really good point.

Closed as completed by PR

https://github.com/notaryproject/notation/pull/827
https://github.com/notaryproject/notation/pull/842
https://github.com/notaryproject/notation/pull/855


Fixed incorrect number in getSummonerSpell2

getSummonerSpell2() was calling data.getSpell1ID() instead of data.getSpell2ID()

Thanks


Syntax error: Solved

That semicolon is actually required to be syntactically correct. The enum has no values and defines two fields. The semicolon marks the end of the values part.

Hey Thanks for the insight, I got it.


Fixes modules/types not being found by typescript from js packages

Description of change

As discussed here #6261 and in the slack channels there have been issues with getting types and modules being recognised properly.
Before we ran into issues of either a module not being recognised or cannot use namespace as type, depending on the different solutions that have been thrown around.
With this change we no longer need to declare modules inside the global.d.ts file (except for settings as this one is weird). We should also get types for packages that are still in javascript.
I've tested this out on my display branch and it fixed all my build issues.
@EvidentlyCube @ivanpopelyshev  this should help with the core/spritesheet packages as well.
Pre-Merge Checklist

[x ] Lint process passed (npm run lint)
[x ] Tests passed (npm run test)

Codecov Report

Merging #6311 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #6311   +/-   ##
=======================================
  Coverage   77.57%   77.57%
=======================================
  Files         186      186
  Lines       10083    10083
=======================================
  Hits         7822     7822
  Misses       2261     2261

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ab9ada6...7a9be0b. Read the comment docs.

Holy Cow!


Add the ability to transfer a name

Solved by https://github.com/ethereum/ens-registrar-dapp/pull/43


[BEAM-9899] Fix some issues around storing schema id on user types

Updates sql_test to require generating user types with an id field, which yields a name collision with the id added to generated NamedTuple types.
~Modifies row_coder to extract values from NamedTuple instances as plain tuples, rather than using getattr. This fixes the above name collision.~ Dropped this change since it's not strictly necessary
Also, store schema id in an attribute called _beam_schema_id rather than id to avoid future name collisions.
Adds logic to set _beam_schema_id for provided (non-generated) user types as well as a test of this logic. This allows us to retrieve the same schema for the type in the future, rather than re-generating it.


allow Value::Bytes to be assigned to Schema::Fixed ...

... as long as the length matches. This seems needed to enable Serde and serde-bytes to provide Fixed values.

Not sure why the tests failed. The all pass when I run locally. It looks like it was an issue with the test system rather than the code.

Not sure why the tests failed. The all pass when I run locally. It looks like it was an issue with the test system rather than the code.

Hello, sorry for the late reply!
Could you try to rebase on master and to run cargo fmt? We had clippy failing in master, but now it should be fixed,  while cargo fmt seems to fail for your branch.

Hello, sorry for the late reply!
Could you try to rebase on master and to run cargo fmt? We had clippy failing in master, but now it should be fixed,  while cargo fmt seems to fail for your branch.

Hello! As of today, I am stepping down as a maintainer of this library for a while (see #174). Please ping @flavray from now on.
Thanks again for your contribution :)

Hello! As of today, I am stepping down as a maintainer of this library for a while (see #174). Please ping @flavray from now on.
Thanks again for your contribution :)

@flavray merge and formatted, as requested.
tests pass locally. we'll see if they are able to run now. there was some permissions error last time.

@flavray merge and formatted, as requested.
tests pass locally. we'll see if they are able to run now. there was some permissions error last time.

v0.13.0 has just been released with this change. 🙂

v0.13.0 has just been released with this change. 🙂


Show all ipv4 addresses when server is running on unspecified address

Ref: https://github.com/nuxt/nuxt.js/issues/2180
It can be confusing that if you run the server with a unspecified address it will always show that its running on localhost. This pr changes that behaviour by listing all ipv4 address the server is listening on.
Example output:

Codecov Report

Merging #2182 into dev will decrease coverage by 0.99%.
The diff coverage is 23.07%.

@@           Coverage Diff           @@
##              dev    #2182   +/-   ##
=======================================
- Coverage   98.04%   97.04%   -1%
=======================================
  Files          15       15
  Lines         970      982   +12
=======================================
+ Hits          951      953    +2
- Misses         19       29   +10

Impacted Files
Coverage Δ

lib/core/nuxt.js
87.5% <23.07%> (-12.5%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9bf73b0...4eff3b9. Read the comment docs.

Hmm, I think it will be best to close this PR as I dont really understand the need for the OPEN statement in the way you describe so I dont think I will be able to change the PR in the way you like. I really thought it was more of a LISTEN confirmation which I am used to get when starting a networking service.
As this is such a small issue I just don't think its worth your or my time to discuss it either. It will take you probably less time to implement it exactly the way you prefer then to explain that way to me. Hope you are ok with that :smile:


ci: added github actions workflow

Here is an example of GH actions workflow run: https://github.com/audrium/reana-message-broker/actions/runs/343641446

Cannot merge due to lack of privileges:
remote: error: GH006: Protected branch update failed for refs/heads/master.
remote: error: At least 1 approving review is required by reviewers with write access.
To github.com:reanahub/reana-message-broker
 ! [remote rejected] master -> master (protected branch hook declined)
error: failed to push some refs to 'git@github.com:reanahub/reana-message-broker'


WiP: Remove fixing duration from bears.json

This PR targets issue #670
Backup files will be used for repairnator.json.

This PR has not been tested at all. I will work on Travis build results until everything it is passing.

Pull Request Test Coverage Report for Build 1914

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.007%) to 54.749%

Totals

Change from base Build 1911:
-0.007%

Covered Lines:
2882

Relevant Lines:
5264

💛  - Coveralls

Apparently it is already good.


Bridgy RSVP pages could use simpler, shorter, DRYer markup

E.g. https://brid-gy.appspot.com/rsvp/facebook/11500459/444025702419735/214611
has:

Tantek Çelik is attending.

Tantek Çelik

  <data class="p-rsvp" value="yes">is attending.</data>

Which could be greatly simplified to:

Tantek Çelik

  <data class="p-rsvp" value="yes">is attending.</data>

This results in the following parsed result:
    {
      "type": [
        "h-entry"
      ],
      "properties": {
        "author": [
          {
            "type": [
              "h-card"
            ],
            "properties": {
              "name": [
                "Tantek Çelik"
              ],
              "url": [
                "https://www.facebook.com/214611"
              ],
              "photo": [
                "https://graph.facebook.com/v2.2/214611/picture?type=large"
              ]
            },
            "value": "Tantek Çelik"
          }
        ],
        "name": [
          "is attending."
        ],
        "rsvp": [
          "yes"
        ],
        "url": [
          "https://www.facebook.com/444025702419735#214611"
        ]
      }
    }

thanks guys!
@kylewm, you might be closer to this than me, so you're welcome to take a stab if you want (and if you agree with the suggestion here). my guess is we'd want to forego the HENTRY template entirely and add a new custom RSVP template, similar to the existing HCARD, but you're welcome to go with whatever you think works best if you want.

i wonder how much of this would also apply to likes, reposts, and maybe even comments. let's see! here are examples of bridgy's current markup for each:

like, https://www.brid.gy/like/facebook/212038/10101025265731053/10100410728079479

  tag:facebook.com,2013:123_liked_by_456

Ellen B likes this.

Ellen B

likes this.

repost, https://www.brid.gy/repost/twitter/benwerd/565302012729122816/565302713685397508

  tag:twitter.com,2013:565302713685397508

Carl Jeffrey retweeted this.

  <time class="dt-published" datetime="2015-02-11T00:14:04+00:00">2015-02-11T00:14:04+00:00</time>

Carl Jeffrey

    retweeted this.

comment, https://www.brid.gy/comment/instagram/snarfed/921288114267399137_420973239/921330190979867628

  tag:instagram.com,2013:921330190979867628

  <time class="dt-published" datetime="2015-02-16T01:45:39">2015-02-16T01:45:39</time>

Tantek Çelik

    How do we deal with spam like that from seaglehelga? Do we need a "report as spam" reply and Bridgy support to pass it on to silos?

ok, looking at those three examples...i think the comment markup is pretty much fine as is, but the like and repost can be refactored similarly to rsvp. here are the results.
like:

Ellen B

  likes this.

repost:

Carl Jeffrey

  retweeted this.

<time class="dt-published" datetime="2015-02-11T00:14:04+00:00">2015-02-11T00:14:04+00:00</time>

@tantek, @aaronpk, @kevinmarks, @kylewm, do those pass your sniff test?

smaller:
like:
Ellen B
likes this.

repost:

  Carl Jeffrey

retweeted this.

<time class="dt-published">2015-02-11 00:14:04+00:00</time>

thanks! i doubt we'll end up with that exact markup, but we'll get as close as we can. good to know this kind of refactoring will work for likes and reposts as well as rsvps.
@kylewm, holler if you're interested in this. if not, no worries, i have bandwidth for at least the next few days or so, i can take it.

we're definitely not at the ideal minimal markup yet, but i've dropped e-content and trimmed p-name from rsvps, likes, and reposts, which i think was the important point here. here are the example responses:

https://www.brid.gy/rsvp/facebook/11500459/444025702419735/214611
https://www.brid.gy/like/facebook/212038/10101025265731053/10100410728079479
https://www.brid.gy/repost/twitter/benwerd/565302012729122816/565302713685397508

and the new parsed mf2:

https://kylewm.com/services/mf2?url=https://brid-gy.appspot.com/rsvp/facebook/11500459/444025702419735/214611
https://kylewm.com/services/mf2?url=https://www.brid.gy/like/facebook/212038/10101025265731053/10100410728079479
https://kylewm.com/services/mf2?url=https://www.brid.gy/repost/twitter/benwerd/565302012729122816/565302713685397508

(repost still has the content, at least for twitter, which is intentional. i'm still iterating on it.)
we'll continue improving the markup, of course, but i'm going to tentatively close this for now. thanks for all the detailed debugging help, everyone!

it's difficult to mark up reposts in a way that presents good name and content to both readers and webmention receivers.
readers would like content="RT @[author] [content]"; webmention receivers want content="reposted this." I continue to be at a loss on how to reconcile these :cry:

It seems that readers would get the contents of the repost from the "repost-of" object, or optionally go fetch the original URL.

@aaronpk so readers that don't have special handling for reposts would just see p-name = e-content =
reposted this.

? That seems reasonable enough, but ideally I want to include a higher fidelity representation even for naive readers.

I see the problem, and I also can't think of a solution that solves both. I guess I feel like it's okay to put the burden on readers to support higher fidelity rendering of reposts, since there will always be far fewer readers than there are websites showing comments.


Seperate CI for focal and jammy. Added humble distro to workflow

Configures two (almost identical) workflows

foxy and galactic to be built on Ubuntu focal
humble and rolling to be built on Ubuntu jammy

Had to exclude include-order from cpplint, as it is expecting different orders on foxy+galactic resp. humble+rolling and I don't want to branch just for this.
Signed-off-by: Arne Nordmann<EMAIL_ADDRESS>

Configures two (almost identical) workflows

foxy and galactic to be built on Ubuntu focal
humble and rolling to be built on Ubuntu jammy

Had to exclude include-order from cpplint, as it is expecting different orders on foxy+galactic resp. humble+rolling and I don't want to branch just for this.

If I see it correctly, the only difference between the two Action workflow files is the Linux distribution. Two files are not necessary. The ROS distributions and the corresponding Linux distributions can be also specified as a list of pairs, cf. https://github.com/boschresearch/fmi_adapter/blob/master/.github/workflows/build_and_test.yml#L16


Ability to step past errors in StreamDeserializer if input is valid JSON

The current behavior of StreamDeserializer is that if it hits an error, that same error is returned in perpetuity which is typically not what you want. (If that is what you want, it is easy to implement that behavior atop what I propose below.)
Suppose we have a stream deserializer that expects to read a stream of Strings. If it receives the input "a" {"b": 0} "c", a reasonable thing to expect as output would be "a" (error) "c" because the second JSON object failed to deserialize to a String.
On the other hand, if the input is not a valid JSON stream I believe the StreamDeserializer should retain its current behavior because there is no way for it to regain its bearings in an invalid JSON stream. For example if the input is "a" {"b{{"c" "d", I would expect the output to be "a" (error) (error) (error...).

The desired behavior:
let data = b"[0] {"k":"v"} [1]";

let de = serde_json::Deserializer::from_slice(data);
let mut stream = de.into_iter::<Vec<i32>>();
assert_eq!(0, stream.byte_offset());

println!("{:?}", stream.next()); // [0]
assert_eq!(3, stream.byte_offset());

println!("{:?}", stream.next()); // type error
assert_eq!(13, stream.byte_offset());

println!("{:?}", stream.next()); // [1]
assert_eq!(17, stream.byte_offset());

Has there been any progress towards implementing the change mentioned in the StreamDeserializer docs?

Note: In the future this method may be changed to return the number of bytes so far deserialized into a successful T or syntactically valid JSON skipped over due to a type error. See serde-rs/json#70 for an example illustrating this.

This is the behavior I was expecting from StreamDeserializer::byte_offset; that it would represent the total number of bytes processed so far, regardless of the result those bytes produced. Although it's not clear to me what the behavior should be for syntactically invalid JSON.

While this could still be useful to implement, I'll describe my workaround as well which may be a better solution. Rather than using the StreamDeserializer to produce values of T directly, so it's doing both parsing and deserialization, I'm just getting Values from the StreamDeserializer so all it's doing is parsing JSON and will always advance if it finds valid JSON. Then later I'm using from_value to convert the JSON into my domain types.

An alternative is to move the T from StreamDeserializer<'_, R, T> down to fn next(&mut self), ie fn next<T>(&mut self) -> Option<Result<T>> so that each individual call to .next() can generate a different type.
This way the user's loop can be to first try de.next::<Vec<i32>>(), and when that fails with Some(Err(...)), they can retry with de.next::<IgnoredAny>(), then go back to de.next::<Vec<i32>>()
This gives the user precise control of how they want to handle the intermediate error. But of course the con is that this means removing the Iterator impl, and thus losing all the combinators and such that users can use right now.

If someone is looking for a workaround, the solution from @H2CO3 on the Rust users forum helped me. It steps past an item in the stream even if it is invalid JSON.


Improved error for timeout

Hi!
Currently, when running in a timeout, for example with the query SELECT sub.ret FROM (SELECT pg_sleep(10), 'OK' AS ret) AS sub, the error returned is {error, closed}.
Do you think it would be possible to return the error {error, timeout} instead?

Great point, let me see.


Notification function not working for Samsung S10

I turned on the notification button and at the beginning, it only notified disconnection, other notification like full bag is not notified. A few days ago, even disconnection notification is not pushed.

Did you make sure that the "Monitor game notifications" switch is on in the app?

Closing due to inactivity.


[Popup] Polyfill

I love the idea of having a native popup and would love to start programming against the API now with a polyfill, which I could remove later on.
Would this be possible?

Hi @newtack, I'm so glad to hear you're excited about the <popup> proposal and that this would be imminently useful for you! I'm porting popup issues over to Open UI, where we'll be incubating the proposal further. So if you'd like to keep tabs on this discussion, it will now be on Open UI #312. Thanks again!


Fix XSS Vulnerability

This is an initial attempt to fix XSS scenarios noted in https://github.com/jbt/markdown-editor/issues/106. If any AllowScriptAccess attributes are found in HTML elements, or if any event handler attributes are set to HTML elements, then the related text will be automatically removed from the CodeMirror textarea. It also alerts the user with a descriptive message on why their added text was filtered out.

@aero31aero I'd like to assign you as a reviewer to this pull request, please let me know if you have any suggestions. I feel like this should be merged in as soon as possible since the current live version of the editor can be exploited with XSS attacks and the link-sharing feature amplifies how dangerous it can be.


fix: fixed incorrect usage of log.exception

Analogous to https://github.com/epam/ai-dial-adapter-vertexai/pull/97

/deploy-review


Update Project Usage Documentation

NOTE: This is an issue that exclusively demands knowledge of Python. Please take this up only if you are comfortable beyond basics
Package to be used: MkDocs Material
Branch for source code: documentation
Please read through the documentation carefully and dedicate some time!
Documentation Requirements:

Welcome [Refer the readme]

About the project
Why Open Source?
Programs we have been a part of
Sneak Peak (Use Gifs)
Team

** Please do beyond just copy-paste.

How to start using it?

Install Git
Setup Git and GitHub Account
Navigate to GitHub Repository
Fork the repo
Clone the repo [ use git clone ..... --depth 1]
Access the local copy
Delete Additional Stuff ( Readme, Readme assets,etc)

Page by Page Changes:

Home Page
Education Page
Project Page
Experience Page
Research Page
Social Media Links
Assets
SEO

^^ Detailed guide on which files to open, where to change the content and for what parts? Use screenshots of the changes!

Publishing the portfolio!

Save the changes
The step by step to push to Github
Tip: On making the repository private if desired
Deployment + Using Custom Domain:
- GitHub Pages
- Netlify
- AWS Amplify

Directions:

Some of the content is already there. We expect you to improve it in any or all ways possible!
Please take only one page at a time to update and state it clearly in the comments below
First, update the source code, check your build output, push to your own gh-pages branch
Then in the PR, compare both the documentation and gh-pages branch (separate PRs but would be counted as a pair)
Attach a screenshot of your changes
Attach a link to your own deploy preview on GitHub pages
Use proper comments in the markdown files

@lavakush07 do give regular updates here and if you face issues ask @smaranjitghose

@lavakush07 do give regular updates here and if you face issues ask @smaranjitghose

Yeah sure..

@lavakush07 Updates?


Scanner unable to validate (almost) all of my artifacts

I was using 1280x720 Windowed as suggested and the scanner was only able to validate 15 out of 328 artifacts.
I've also tried different Scan Delay settings with no difference.

Have you tried restarting genshin at the same resolution? There are known issues with resizing the window that messes up the UI a bit.

I have done that just now, closed and opened Genshin at 1280x720 but It had the same result.
It's weird that it only works for a very few artifacts.

You're right that it is strange that it can scan some artifacts but not nearly a significant percent. I will have to write some better error output to be able to tell exactly what's causing the validation to fail.

Hi i had the same problem and on Genshin Optimizer i have notice that only artefact with only FLAT sub stat, none of them have  a percent stat. I don't know if it help.

@Ame55 this seems to be a different issue. Please open a new issue. Additionally if you can provide a screenshot of at least one of the artifacts in-game that should have % substats and how it appears in GO it would be much appreciated.

I do think it's the same issue, I also get Unable to validate information for artifact #329 for every artifacts that have a % substats.

During the scan it didn't show anything in the output artifacts windows, so don't know what causing the validation to fail.

Sorry, I misspoke, I meant that like Paws26, and now Dranixx, only a small part of my artifacts were scanned. The Error Unable to validate information for artifact #

And that after transferring the data to GO, I noticed that the scanned artifacts only had flat substat.

I apologize for not having correctly explained the problem previously.

same here, worked fine with 1.0. since 1.1 substats are missing

I managed to get all scanned after pulling the development version from git, building it with Visual Studio and then replacing v1.1 version files with the newly built files. However, I can't import anything to seelie.me even thou the json-file has all the data in it.

Hi there, this issue is hopefully resolved in the v1.1.1 prerelease. If it has been, please let us know so we can close this issue.
@tolukolvi : There are two places you can import to seelie.me, one in the Inventory panel and one in the Settings panel. I believe you need to use the Inventory panel's import button and use the GOOD option.

I am going to close this issue as one user has reported it being fixed and the other users in the thread have not replied in a while. Please feel free to reopen this issue or to open a new issue should any problems come up.


Incorrect redirection on order packing.

Expected behavior
Upon packing an order, the user should be directed to the shipping label or packing slip tab within the current session.
Steps to reproduce the issue
Log in to the App having view app permission.
Go to In progress tab and select an order to pack.
Initiate the packing process.
Observe the redirection behavior.
Environment details
App Version: V2.0.1
Testing Environment: UAT
Additional information
https://jam.dev/c/6eafb980-e119-4882-8123-0b7483e4afaa

@Adameddy26 Please verify this, its already in production and was part of https://github.com/hotwax/fulfillment/releases/tag/v2.0.2

Verified and working fine in release v2.10.4. Hence, closing this ticket.


Translations - Add i18n support

In which part of the interface would this feature applies?
[x] 1. Load your data
[x] 2. Choose a chart
[x] 3. Mapping
[x] 4. Customize
[x] 5. Export
Is your feature request related to a problem? Please describe.
Any plans to add i18n support? I would be available to contribute with translations.
Describe the solution you'd like
Possible to select language on a drop-down.

Hi, that's something we want to add but we don't have a roadmap yet. We will keep you posted


Custom Center with Edge Utils

First off really I like the added edge utils. This makes creating custom edges much easier.
For my use case, I would like to be able to customize the center of the edge specifically for the smooth step edge. However, the getSmoothStepPath doesn't accept a custom centerX and centerY. Could this method be changed to accept these values?

@moklick Do you have advice here? Really would like to use the custom edge utils with a custom center.

Hey. I think we can add this option.

released in v5.11.1