Monitor showing offline but its online

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
I have the problem, that all of my monitors except 1 showing as offline. i can reach the website normally.
Can someone help me by that?
One Monitor showing "error request aborted"
The other two showing "timeout of 48000ms exceeded"
🐻 Uptime-Kuma Version
1.16.1
💻 Operating System and Arch
Debian 10 x86_64
🌐 Browser
Microsoft Edge 103.0.1264.37
🐋 Docker Version
20.10.17
🟩 NodeJS Version
No response

You mean with the ping command? if yes, yes. i can ping every server

I got the same exact problem every morning at 5-6 AM for an hour. It says that it's down, but it's not. With the same error "timeout of 48000ms exceeded". Usually there are 2 websites, but it happens to be only 1 or 3 sometimes. I've increased the Heartbeat Interval and Retries, but no luck. Anyone can help?

I got the same exact problem every morning at 5-6 AM for an hour. It says that it's down, but it's not. With the same error "timeout of 48000ms exceeded". Usually there are 2 websites, but it happens to be only 1 or 3 sometimes. I've increased the Heartbeat Interval and Retries, but no luck. Anyone can help?

We got the same thing last night for maybe 40-50 Websites.

Now my Discord-Webhook showing sometimes, that the Services are online but then it says there are offline.

Same issue here, see: https://github.com/louislam/uptime-kuma/issues/1621

Same issue here, suddenly we are getting "timeout of 48000ms exceeded" on most of the checks.
Anyone has any clue?

Same issue here, suddenly we are getting "timeout of 48000ms exceeded" on most of the checks.
Anyone has any clue?

This may help:
https://github.com/louislam/uptime-kuma/wiki/Troubleshooting

Hi I have tried to run the cmd without apparently any issue.
Best regards,
Francesco.
@.*** ~]$ curl https://google.com

301 Moved
301 Moved
The document has moved
here.

@.*** ~]$ ping google.com
PING google.com (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from ams16s32-in-f14.1e100.net (<IP_ADDRESS>): icmp_seq=1 ttl=53 time=2.77 ms
64 bytes from ams16s32-in-f14.1e100.net (<IP_ADDRESS>): icmp_seq=2 ttl=53 time=2.52 ms
64 bytes from ams16s32-in-f14.1e100.net (<IP_ADDRESS>): icmp_seq=3 ttl=53 time=2.80 ms
64 bytes from ams16s32-in-f14.1e100.net (<IP_ADDRESS>): icmp_seq=4 ttl=53 time=2.36 ms
64 bytes from ams16s32-in-f14.1e100.net (<IP_ADDRESS>): icmp_seq=5 ttl=53 time=2.36 ms
64 bytes from ams16s32-in-f14.1e100.net (<IP_ADDRESS>): icmp_seq=6 ttl=53 time=2.42 ms
64 bytes from ams16s32-in-f14.1e100.net (<IP_ADDRESS>): icmp_seq=7 ttl=53 time=2.63 ms
64 bytes from ams16s32-in-f14.1e100.net (<IP_ADDRESS>): icmp_seq=8 ttl=53 time=2.67 ms
64 bytes from ams16s32-in-f14.1e100.net (<IP_ADDRESS>): icmp_seq=9 ttl=53 time=2.33 ms
^C

On 2 Aug 2022, at 12:18, Louis Lam @.***> wrote:
Same issue here, suddenly we are getting "timeout of 48000ms exceeded" on most of the checks.
Anyone has any clue?
This may help:
https://github.com/louislam/uptime-kuma/wiki/Troubleshooting https://github.com/louislam/uptime-kuma/wiki/Troubleshooting
—
Reply to this email directly, view it on GitHub https://github.com/louislam/uptime-kuma/issues/1848#issuecomment-1202295726, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFCDTMMWIQ2MPJZW2KPTRX3VXDYYDANCNFSM52CMU2PA.
You are receiving this because you commented.

Yeah, that works for me as well. It's really that Uptime Kuma gets bogged down, and it happens more frequently when the container is running for a longer period of time. I've not found anything useful in the logs either, it just times out.

30s monitor (both monitors have the same settings and site, except for the interval):

1m monitor:

Same issue here.
When I try to ping the url from within the docker container, as suggested here, I also do get a timeout.
BUT... the same website can be reached and pinged from other systems AND I can still ping other monitored websites from within the Uptime Kuma docker container, so it's not a general networking issue with the container / server / hosting.
So it seems something is getting bogged down in Uptime Kuma itself.
Note
Most of our monitored websites are behind Cloudflare, but some are not.
And until now I only experienced the issue with websites that are not behind Cloudflare.
Though not all those "non-cloudflare" websites are affected at the same time.
At the moment 2 of the non-cloudflare websites are affected, but several weeks ago we had the same issue with one of those 2 websites (and another one), but not the other.
We also see different errors prior to the timeout issues:
140059089770432:error:1408F119:SSL routines:ssl3_get_record:decryption failed or bad record mac:../deps/openssl/openssl/ssl/record/ssl3_record.c:677
and
getaddrinfo EAI_AGAIN <hostname>

Same issue here. When I try to ping the url from within the docker container, as suggested here, I also do get a timeout.
BUT... the same website can be reached and pinged from other systems AND I can still ping other monitored websites from within the Uptime Kuma docker container, so it's not a general networking issue with the container / server / hosting.
So it seems something is getting bogged down in Uptime Kuma itself.
Note Most of our monitored websites are behind Cloudflare, but some are not. And until now I only experienced the issue with websites that are not behind Cloudflare. Though not all those "non-cloudflare" websites are affected at the same time. At the moment 2 of the non-cloudflare websites are affected, but several weeks ago we had the same issue with one of those 2 websites (and another one), but not the other.
We also see different errors prior to the timeout issues:
140059089770432:error:1408F119:SSL routines:ssl3_get_record:decryption failed or bad record mac:../deps/openssl/openssl/ssl/record/ssl3_record.c:677
and
getaddrinfo EAI_AGAIN <hostname>

Hey, we "fixed" that issue by simply not using docker. But sometimes we are still getting the timeout error but not as often as using docker.

Please don't close this issue as it is still not fixed!

Same issue here. When I try to ping the url from within the docker container, as suggested here, I also do get a timeout.
BUT... the same website can be reached and pinged from other systems AND I can still ping other monitored websites from within the Uptime Kuma docker container, so it's not a general networking issue with the container / server / hosting.
So it seems something is getting bogged down in Uptime Kuma itself.
Note Most of our monitored websites are behind Cloudflare, but some are not. And until now I only experienced the issue with websites that are not behind Cloudflare. Though not all those "non-cloudflare" websites are affected at the same time. At the moment 2 of the non-cloudflare websites are affected, but several weeks ago we had the same issue with one of those 2 websites (and another one), but not the other.
We also see different errors prior to the timeout issues:
140059089770432:error:1408F119:SSL routines:ssl3_get_record:decryption failed or bad record mac:../deps/openssl/openssl/ssl/record/ssl3_record.c:677
and
getaddrinfo EAI_AGAIN <hostname>

Make sure you are not using Alpine based image.
For SSL error,  please double check http:// vs https://

Same issue here. When I try to ping the url from within the docker container, as suggested here, I also do get a timeout.
BUT... the same website can be reached and pinged from other systems AND I can still ping other monitored websites from within the Uptime Kuma docker container, so it's not a general networking issue with the container / server / hosting.
So it seems something is getting bogged down in Uptime Kuma itself.
Note Most of our monitored websites are behind Cloudflare, but some are not. And until now I only experienced the issue with websites that are not behind Cloudflare. Though not all those "non-cloudflare" websites are affected at the same time. At the moment 2 of the non-cloudflare websites are affected, but several weeks ago we had the same issue with one of those 2 websites (and another one), but not the other.
We also see different errors prior to the timeout issues:
140059089770432:error:1408F119:SSL routines:ssl3_get_record:decryption failed or bad record mac:../deps/openssl/openssl/ssl/record/ssl3_record.c:677
and
getaddrinfo EAI_AGAIN <hostname>

Make sure you are not using Alpine based image.

I'm not since I'm running Uptime Kuma with Cloudron, which uses Ubuntu based docker images

For SSL error, please double check http:// vs https://

All sites in Uptime Kuma are configured with https, so there shoudn't be any reason for Uptime Kuma trying to access the sites over http.

For me still happening

For me still happening

Yes it still happening to me

Getting this error when trying to send a webhook, as in https://github.com/louislam/uptime-kuma/issues/1697, running via pm2

Start to happen to me after update the docker image.

Same issue here, suddenly we are getting "timeout of 48000ms exceeded" on most of the checks.
Anyone has any clue?

This may help: https://github.com/louislam/uptime-kuma/wiki/Troubleshooting

No curl installed in docker

After stop docker, pull new imagem and start docker.

ping and curl working from inside the container, nothing is working from this.

I can confirm that issue. the ping works just fine, but the http(s)-typ does not work and I'm getting the error getaddrinfo EAI_AGAIN.

latest official docker image
behind nginx reverse proxy (nginx proxy manager with the recommend settings)
curl is not available inside die container and can not installed like in this article (https://github.com/louislam/uptime-kuma/wiki/Troubleshooting)
tried different methods to install it, but no working solution

same here using the latest image.

I have the same issue for months.. At first I thought this issue from my VPS provider, but after I had big fights with multiple VPS providers, I found out it's only happening from Uptime Kuma.
I think I'll just look for another tool and apologize to the VPS providers!

Just a ping- its still a problem.

But, container logs, do tell me to report it. So. Here it is.

Deleting and recreating the monitor did nothing to help.
However, after deleting the pod, and letting it recreate, monitor did start working.

stalebot, really?

Stalebot is active on all issues using the help template.
=> Closing stale help requests is by design because often people don't answer or issues are not reproduce or we fix things in further releases but forget to close them leading to immortal zombies.
Until now this issue has not really provided a minimally reproducible example (see https://stackoverflow.com/help/minimal-reproducible-example)
If somebody provides such an example I can have a look, but currently this issue doesn't really give enough context to debug against.
There are multiple factors which can lead to timeouts.

I had the problem yesterday, and finally found what occurs…
After a maintenance reboot from our provider, IPv4 remains the same, but IPv6 changed. I would have never seen this change if uptime kuma didn't alert.
I don't know why yet, but my uptime kuma check https services via IPv6 while my desktop ISP still only works with IPv4.
Updating my DNS with the new IPv6 solve the problem.

Same issues here

Same issue here. Was stable for 16-7 months using Node+PM2, switching to Docker was stable for 2 months and then started showing all sites down every 4-10 hours. Was not able to find anything in the container logs to troubleshoot. We're planning to switch back to just running the Node app.


Bug ?

After update:
 when i try to download some documentation, the neovim is literally freezed, and when the docs are downloaded, it then goes normal way(such problems were not encountered before)

if i put some documentation for example python or lua, like this ensure_installed = { "lua", "python" }, then docs are not dowloaded and i get an error that documentation doesnt exist for this filetypes, but the problem i guess is that the docs comes with versions, while i put only the filetype name.

With the update docs are now built at download time and writing files is done synchronously and blocks the input, you may want to download larger docs in headless mode now. Maybe I can create a separate program for that in the future instead of depending on neovim for package management.
You need to specify the version for the lua and python docs


[Macros] Attached macro expasnions return single string

Move collapse(expansions:for:attachedTo:) to SwiftSyntaxMacroExpansion
SwiftSyntaxMacroExpansion.expandAttachedMacro() now perform collapsing
SwiftSyntaxMacroExpansion.expandAttachedMacroWithoutCollapsing() to keep old behavior
IPC request 'getCapability' now sends the host protocol version
Unified IPC response 'macroExpansionResult' that returns single string for both 'expandFreestandingMacro' and 'expandAttachedMacro'
Compiler accepts old 'expandFreestandingMacroResult' and 'expandAttachedMacroResult' to keep compatibility
Plugins check the compiler's protcol version to see if it suppports 'macroExpansionResult', and fall back to old behavior if necessary

Also:

Correct PluginMessage.MacroRole definition mismatch freeStandingDeclaration -> declaration

https://github.com/apple/swift-syntax/pull/1845
@swift-ci Please test

https://github.com/apple/swift-syntax/pull/1845
@swift-ci Please smoke test

https://github.com/apple/swift-syntax/pull/1845
@swift-ci Please smoke test


Clicking dropdown menu trigger intermittently toggles nearby text selection on page

Bug report
Current Behavior
In the dropdown demo page, if you quickly click the menu trigger button, it seems to propagate as a double-click to the underlying page and selects nearby text (in this case all of the text in the block immediately above the demo).
Admittedly this may be a contrived and atypical user interaction, but I was able to inadvertently trigger it enough times in my own application to notice it.
Short video attached.
Expected behavior
Clicking the menu trigger button should suppress such events.
Reproducible example
https://www.radix-ui.com/docs/primitives/components/dropdown-menu
Suggested solution
?
Additional context
Tested with Chrome on Linux, macOS and Windows, all with same result.
Issue did NOT happen for me on Firefox.
Seems similar maybe to #1658 .
Your environment

Software
Name(s)
Version

Radix Package(s)
react-dropdown-menu
2.0.5

React
n/a
18.2.0

Browser
Chrome
Version 114.0.5735.198 (Official Build) (64-bit)

Assistive tech

Node
n/a
16.14.2

npm/yarn
npm
8.5.0

Operating System
Linux Mint, macOS, Windows 11
latest, fully updated

https://github.com/radix-ui/primitives/assets/1385879/7da07dac-6eaf-494d-8ce1-719e7c0c334d

Admittedly this may be a contrived and atypical user interaction.

Agreed, I'll take a look if it can be fixed easily, but otherwise might not be worth it.

Same issue here.
https://github.com/radix-ui/primitives/assets/3471836/da503b35-7065-4e6b-bd38-03b83e373581


Cannot find name Constructor

I'm getting about 90 of these errors, all originating from this library in node_modules.  I don't know why they're being generated (tsc shouldn't be tying to compile anything in node_modules) but I do note that I can't seem to find a "Constructor" defined anywhere in the library - so I don't know what's going on.

Thanks for the heads up! I've been working on this library within itself and haven't tried importing it from the outside for a while. I'll have to setup an automated test to make sure this doesn't happen again (Opened #33).
I was able to reproduce the error you mentioned and this should be fixed in 0.28.1. Let me know if this fixes the problem for you or if you have any other issues.

Yes, this does appear to be fixed in 0.28.1!  I'm having new issues now which may be by design.  I'll open a separate issue for each of those.


Support for integer/data textures in WebGL2 and WebGPU

Add support for many of the signed/unsigned integer formats supported by WebGL 2.
Integer textures are very useful for emulating compute shaders in WebGL using data textures, voxel rendering and more.
With that said, new formats do require a few lines of repetitive definition code - 573 net lines here. I won't be offended if you decide that the potential use-cases are too niche to justify the added weight.
Happy Holidays!
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.
Example:
https://github.com/playcanvas/engine/assets/453578/12953eca-0dcd-4825-a5a3-8b1b2f335d32

Just to add here - I'd love to have an engine example here showing the use of some of these. Any thoughts on this @liamdon ?

Thank you for all the comments @mvaligursky! I'll address the issues above.
Regarding the engine example: yes, I thought about doing this but wanted to make sure you were OK with the idea before investing time in an example. I'll add an example to this PR, but likely over the weekend.

Thanks for the additional comments @mvaligursky. It was a good idea to implement the example as I found some issues with transpiling shaders to WGSL, which are now fixed.
For these compute-ish use-cases, it's very useful to be able to specify the output type! So you saw the change I made there - I will update it to better support the MRT case.
A future PR I've been thinking about is a higher-level compute API that operates on any TypedArray, with shaders that have a GLSL-ish syntax similar to other PlayCanvas shaders. On WebGPU, this could be transpiled to a compute shader, while on WebGL it will use a fragment shader / renderTarget approach similar to my example. The selection of a texture format would be transparent to the user, they will just pass in a TypedArray and we will select an approach and create a texture based on the type and stride options.

@mvaligursky updated to use an options object in shader creation, and allow specifying an output type per color attachment.
See relevant changes:
shader-utils.js
gles3 chunk
webgpu chunk
Note that in the webgpu header we're defining a default outType for all targets inside the preprocessor because unlike in gles3, these targets are not wrapped with a directive. In theory they should always be defined by ShaderUtils.createDefinition but I don't want shader compilation to fail if they're created via some other path.

@mvaligursky I think the only thing left to resolve is the question about the UI naming for PIXELFORMAT_.
I shared my perspective here, which is that we should keep UI for now as it maps to the OpenGL format names. However, as the engine maintainer, you have the final call; if you prefer the single character naming I am happy to update it.

Nice updates, few more small changes and we're done.

A future PR I've been thinking about is a higher-level compute API that operates on any TypedArray

I'd say this is something that would be outside of the core engine. Possibly a separate repo, or at least part of the /extras. But sounds very exciting! Existing issue on this: https://github.com/playcanvas/engine/issues/4769
It would be good to start discussion on this before writing code to see where this would belong I think.

@mvaligursky addressed all your comments above - I really appreciate the rounds of review on this, the PR is much better now!
Want to make sure you saw my note on comparison here. I think we should leave this as-is and potentially change it if we discover that we're setting up our samplers incorrectly elsewhere. For now, we are definitely covering the most common use-case of using texelFetch with an int or uint texture.

Want to make sure you saw my note on comparison here.

I'm not sure you added the note. Perhaps it'd be the best to add a comment in the code to explain.

@willeastcott No problem! Do you use a formatter or IDE extension that flags those for you, or just eyeballing?

Just eyeballed it. 😄

Awesome, just enabled that. There's always a new VSCode feature to discover!
I realized that my earlier replies/comments to you were not submitted @mvaligursky, I did not toggle on the Github review mode 🤦♂
Here is that last comment I was talking about, but I also added similar comment in the code to clarify the use of 'comparison'. After we've merged this PR, I will experiment with using/supporting other types of samplers with integer textures. But, I think the PR as-is will cover the most common way they are used.
Let me know if I missed anything else, otherwise I think we are good to go!

The only remaining issue for me is to change the pixel format 'UI to U, to keep the type a single letter.
We already have F for float (PIXELFORMAT_RGB16F), we're adding I for int types, and I'd prefer U for uint types instead of UI.
You mention we can later rename those .. but lets avoid that, as deprecating them and having new names means the engine needs to support both, and it creates a possibly more work to fix examples, Editor projects and similar.
Single letter type is something I'm already using for uniform types.
Naming compatibility with WebGL naming is not very relevant in our API.

Done - was not a strong opinion against that from me, but I wanted to present the 'WebGL' reasoning in case it was important. We are now using a single character for all the type suffixes.


add juxt

https://clojuredocs.org/clojure.core/juxt

Thanks for the contribution. Some changes here and there but I can see this being merged as soon as any issues with it are ironed out.

All done!

Looks great! Sorry it took me a whole day to get back to you!

No problem. I'm looking forward to contributing more to this project.
Thanks!


test: add secrests tests

Description
Adds tests to clusters secrets.
Related Issue
Fixes #2512
Checklist before merging

[x] Test, docs, adr added or updated as needed
[x] Contributor Guide Steps followed

The functions UpdateZarfManagedImageSecrets and UpdateZarfManagedGitSecrets should be moved to tools or refactored to return an error.


Move development into main branch

Is your feature request related to a problem? Please describe.
OpenSearch projects use main branch for future development. Let's get ourselves out of develop?
Describe the solution you'd like
Merge develop into main and delete it.

@dblock
Hi, could you suggest a better strategy for this change.

Direct merge from develop to main has conflicts.
I tried to send a PR from my personal branch which resolves the above conflict(simply just take everything in develop), but the PR doesn't allow create a merge commit, only squash is allowed. By doing this, it seems we lose all the commits history?

I tried to delete the main branch and cut a new branch from develop, but since they both are protected branch, it doesn't allow me do delete
I tried to force push to main, again protected branch doesn't allow force push

I'd suggest that maybe you or someone has the permission, to just delete main branch because there's nothing useful there anyways. Then I'll cut a branch named main from develop.

I have access, so enabled the merge and rebase options for now. Let me know when you want them switched back.
(Personally I dislike squash only, but this is org's choice rn.)


[Flake] Failed to update config to component deployed

[kind/bug]

What versions of software are you using?

Run pr (on two test node) in OpenShift CI. I have seen this even on single test node.
Actual behavior
Flakey
Expected behavior
Should not be flakey
Any logs, error output, etc?
Reference  - https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/openshift_odo/1920/pull-ci-openshift-odo-master-integration/1126

Same error i can reproduce it on my local setup on the cluster of 3.x that i have been using since 2 days, however on a fresh cluster the chance of hitting the issue is very less (1 out of 10).
Same observation i can find on 4.* cluster on aws too.

/priority high
/kind flake
/area testing
/remove-kind bug

Some more same failure in CI - https://storage.googleapis.com/origin-ci-test/pr-logs/pull/openshift_odo/2001/pull-ci-openshift-odo-master-integration/1241/build-log.txt

Reproduced in CI - https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/openshift_odo/1954/pull-ci-openshift-odo-master-integration/1061
odoURLIntegration Listing urls
  should list appropriate URLs and push message
  /go/src/github.com/openshift/odo/tests/integration/url_test.go:41
 Created dir: /tmp/475069174
Creating a new project: qllqkgdnzd
Running odo with args: [project create qllqkgdnzd -w -v4]
[odo] I0730 05:19:17.632597   31253 preference.go:116] The configFile is /tmp/475069174/config.yaml
[odo] I0730 05:19:17.632680   31253 occlient.go:486] Trying to connect to server api.ci-op-mvjt0q5q-f09f4.origin-ci-int-aws.dev.rhcloud.com:6443
[odo] I0730 05:19:17.651839   31253 occlient.go:493] Server https://api.ci-op-mvjt0q5q-f09f4.origin-ci-int-aws.dev.rhcloud.com:6443 is up
[odo] I0730 05:19:17.754536   31253 occlient.go:416] isLoggedIn err:  <nil>
[odo]  output: "developer"
[odo] I0730 05:19:17.754579   31253 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: context
[odo] I0730 05:19:17.754591   31253 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: context
[odo] I0730 05:19:17.754648   31253 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: app
[odo] I0730 05:19:17.754663   31253 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: project
[odo] I0730 05:19:17.789847   31253 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: app
[odo] I0730 05:19:17.789873   31253 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: output
[odo] I0730 05:19:17.789878   31253 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: component
[odo]  •  Waiting for project to come up  ...
[odo]  ✓  Waiting for project to come up [492ms]
[odo]  ✓  Project 'qllqkgdnzd' is ready for use
[odo]  ✓  New project created and now using project : qllqkgdnzd
[odo] I0730 05:19:18.291503   31253 odo.go:70] Could not get the latest release information in time. Never mind, exiting gracefully :)
Running odo with args: [create nodejs --context /tmp/475069174 --project qllqkgdnzd zthnim --ref master --git https://github.com/openshift/nodejs-ex --port 8080,8000]
[odo]  •  Validating component  ...
[odo]  ✓  Validating component [61ms]
[odo] Please use `odo push` command to create the component with source deployed
[odo]
Running odo with args: [push --context /tmp/475069174]
[odo] Validation
[odo]  •  Checking component  ...
[odo]  ✓  Checking component [100ms]
[odo]  •  Checking component  ...
[odo]  ✓  Checking component [34ms]
[odo]
[odo] Configuration changes
[odo]  •  Creating component  ...
[odo]  ✓  Initializing component
[odo]  •  Triggering build from git  ...
[odo]  ✓  Triggering build from git [121ms]
[odo]  •  Waiting for build to finish  ...
[odo]  ✓  Waiting for build to finish [2m]
[odo]  •  Deploying component zthnim  ...
[odo]  ✓  Deploying component zthnim [18s]
[odo]  ✓  Creating component [2m]
[odo]  •  Applying configuration  ...
[odo]  ✓  Applying configuration [12215ns]
[odo]
[odo] Pushing to component zthnim of type git
[odo]  ✓  Changes successfully pushed to component
Running odo with args: [url list --context /tmp/475069174]
[odo]  ✗  no URLs found for component zthnim in application app
Running odo with args: [url create laulo --port 8080 --context /tmp/475069174]
[odo]  ✓  URL laulo created for component: zthnim
[odo]
[odo] To create URL on the OpenShift Cluster, please use `odo push`
Running odo with args: [url list --context /tmp/475069174]
[odo] Found the following URLs for component zthnim in application app:
[odo] NAME      IN CONFIG     URL                          PORT
[odo] laulo     Present       <not created on cluster>     8080
[odo] To create URLs on the OpenShift Cluster, please use `odo push`
Running odo with args: [push --context /tmp/475069174]
[odo] Validation
[odo]  •  Checking component  ...
[odo]  ✓  Checking component [199ms]
[odo]  •  Checking component  ...
[odo]  ✓  Checking component [27ms]
[odo]
[odo] Configuration changes
[odo]  •  Applying configuration  ...
[odo]  •  Triggering build from git  ...
[odo]  ✓  Triggering build from git [327ms]
[odo]  •  Waiting for build to finish  ...
[odo]  ✓  Waiting for build to finish [1m]
[odo]  ✗  Failed To Update Config To Component Deployed
[odo]  ✗  Applying configuration [1m]
Deleting project: qllqkgdnzd
Running odo with args: [project delete qllqkgdnzd -f]
[odo] This project contains the following applications, which will be deleted
[odo] Application app
[odo] This application has following components that will be deleted
[odo] component named zthnim
[odo] No services / could not get services
[odo]  •  Deleting project qllqkgdnzd  ...
[odo]  ✓  Deleting project qllqkgdnzd [6s]
[odo] Deleted project : qllqkgdnzd
Deleting dir: /tmp/475069174
 • Failure [206.767 seconds]
odoURLIntegration
/go/src/github.com/openshift/odo/tests/integration/url_test.go:13
  Listing urls
  /go/src/github.com/openshift/odo/tests/integration/url_test.go:40
    should list appropriate URLs and push message [It]
    /go/src/github.com/openshift/odo/tests/integration/url_test.go:41
     No future change is possible.  Bailing out early after 87.915s.

    Expected
        <int>: 1
    to match exit code:
        <int>: 0
     /go/src/github.com/openshift/odo/tests/helper/helper_run.go:29

@amitkrout This might be the underlying cause
✗  Waiting for build to finish [30s]
 ✗  Applying configuration [33s]
I0814 13:00:16.602706   13009 cmdutils.go:26] Error:
unable to build the component nodejs-nodejs-ex-ayat: unable to follow logs for nodejs-nodejs-ex-ayat-app-1: unable get build log nodejs-nodejs-ex-ayat-app-1: unable to wait for build nodejs-nodejs-ex-ayat-app-1 to run: timed out waiting for the condition
 ✗  Failed To Update Config To Component Deployed

Are we getting any closer to resolving this?
I found another instance of this in https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/openshift_odo/2053/pull-ci-openshift-odo-master-integration/1440  (https://github.com/openshift/odo/pull/2053)

@amitkrout This might be the underlying cause
✗  Waiting for build to finish [30s]
 ✗  Applying configuration [33s]
I0814 13:00:16.602706   13009 cmdutils.go:26] Error:
unable to build the component nodejs-nodejs-ex-ayat: unable to follow logs for nodejs-nodejs-ex-ayat-app-1: unable get build log nodejs-nodejs-ex-ayat-app-1: unable to wait for build nodejs-nodejs-ex-ayat-app-1 to run: timed out waiting for the condition
 ✗  Failed To Update Config To Component Deployed

Not always, even after successful build the same error appears
[odo]  ✓  Waiting for build to finish [50s]
[odo]  ✗  Failed To Update Config To Component Deployed
[odo]  ✗  Applying configuration [50s]

@mik-dass I am seeing this issue consistently on latest 4.2 OpenShift nightly build. IMO this is no more a flake as on 4.2 nightly build the failure rate is 100%.
Can you please take this issue on high priority basis

@amitkrout https://github.com/openshift/odo/pull/2016#issuecomment-532100708 not sure if this is the same issue as this but this is what I have discovered till now. I am constantly hitting this issue on a 4.2 cluster. Please share the logs of the failures.

@amitkrout #2016 (comment) not sure if it is the same issue as this but this is what I have discovered till now. I am constantly hitting this issue on a 4.2 cluster. Please share the logs of the failures.

It is the same error https://github.com/openshift/odo/pull/2016#issuecomment-532100708. However running the push again successfully pushes the changes to the component.

However running the push again successfully pushes the changes to the component.

Yes I am seen this same behavior too

/reopen

@mik-dass The issue is still persist. It seems you need some more luck in triage this ;), pr #2167 doesn't help atm.

@mik-dass The issue is still persist. It seems you need some more luck in triage this ;), pr #2167 doesn't help atm.

Please point me to the logs

@mik-dass The issue is still persist. It seems you need some more luck in triage this ;), pr #2167 doesn't help atm.

Please point me to the logs

@mik-dass You can find the logs from the above references. Or you can go to the pr history and there you need to find out such failure logs

@amitkrout what reference? you just pointed to #2167  where all the test passed

https://github.com/openshift/odo/pull/2271#issuecomment-540949830
Looking at the logs, it seems to time out after 5 min
[odo]  ✗  Applying configuration [5m]
[odo]  ✗  Failed To Update Config To Component Deployed

That is the timeout time for the WaitAndGetDC() function https://github.com/openshift/odo/blob/599683da41194b64b3ec98e187300bef7701b23a/pkg/occlient/occlient.go#L91. So I believe it is a timeout issue while waiting for the DC.

https://travis-ci.com/openshift/odo/jobs/258102894#L932
Here the problem occurred in the function FollowBuildLog() while setting up the RestClient


Fix to work with latest Hugo

Many Hugo functions and variables have changed case since this last
worked, e.g. safeHtml to safeHTML)
Hugo now requires {{< >}} instead of {{% %}} around shortcodes that
output raw HTML
Fixes #3.

I fixed it, sorry haven't seen your pull request :/


Add conversion ops between batch_bool<float, N> and batch_bool<int, N>

They are exactly the same so we could add implicit conversions, which would make things like mixed select statements easier to use.

We already have the bool_cast free functions for float <-> int32 and double <-> int64 conversions.


[WIP] Refactor move scheme

This is the last bit of cleanup so we can lock down the API for 1.0.
It creates a new class MoveAcceptanceAnalysis, which will replace most of the code in scheme.move_summary and allows removal of related (scheme.move_acceptance, scheme._move_acceptance_line). Also, this new class is tested, unlike the old stuff.
I'm still finishing up the tests, but the code should otherwise be ready for review. I'd like to merge this on Tuesday, so @jhprinz,  please review ASAP if you plan to review.

I've confirmed that this gives the same results as the previous version with both normal TIS simulations and SRTIS simulations (which behave differently because of the null moves). This was done by running the notebooks (but not committing the results).
Since the text is the same in both cases, I'm going to replace the old methods with something that calls the new methods.

Replaced the old version with the new. This is now definitely ready for review.


Writing smaller column batches causes a panic

Describe the bug
I am trying to write a parquet file using the SerializedColumnWriter API (I'm not using RecordBatches), and having issues when trying to write smaller batches to a column.
Schema:
            ....
            required float spectrum_q;
            required float peptide_q;
            required float protein_q;
            optional group reporter_ions (LIST) {
                repeated group list {
                    optional float element;
                }
            }

Each row of my data may or may not have a vector of floats associated with it (reporter_ion.peaks). Ideally, I could iterate throughthe rows, and write to the column in small batches (~16 values at a time, or 1 null value for an empty row).
    let mut scan_map = HashMap::new();

    for r in reporter_ions {
        scan_map.entry((r.file_id, &r.spec_id)).or_insert(r);
    }

    // Caller guarantees `reporter_ions` is not empty
    let channels = reporter_ions[0].peaks.len();

    // https://docs.rs/parquet/44.0.0/parquet/column/index.html
    let def_levels = vec![316; channels];
    let mut rep_levels = vec![1i16; channels];
    rep_levels[0] = 0;

    let col = column.typed::<FloatType>();
    for feature in features {
        if let Some(rs) = scan_map.get(&(feature.file_id, &feature.spec_id)) {
            col.write_batch(&rs.peaks, Some(&def_levels), Some(&rep_levels))?;
        } else {
            col.write_batch(&[0.0], Some(&[1]), Some(&[0]))?;
        }
    }
    column.close()?;

But, this causes a panic
thread 'main' panicked at 'assertion failed: `(left == right)`
  left: `79`,
 right: `0`', /home/michael/.cargo/registry/src/index.crates.io-6f17d22bba15001f/parquet-44.0.0/src/util/bit_util.rs:272:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Aborted

However, if I concatenate all of the data/definition levels/repetition levels together, and then perform one write, it succeeds:
    let mut data = Vec::new();
    let mut def_levels = Vec::new();
    let mut rep_levels = Vec::new();

    for feature in features {
        if let Some(rs) = scan_map.get(&(feature.file_id, &feature.spec_id)) {
            data.extend(rs.peaks.iter().copied());
            def_levels.extend(std::iter::repeat(3).take(channels));
            rep_levels.extend(
                std::iter::once(0)
                    .chain(std::iter::repeat(1))
                    .take(channels),
            );
        } else {
            data.push(0.0);
            def_levels.push(1);
            rep_levels.push(0);
        }
    }
    col.write_batch(&data, Some(&def_levels), Some(&rep_levels))?;

    column.close()?;

I'm not sure if this is a bug, or expected behavior (or a mistake on my end). It looks like there's an internal write_mini_batch function, is it possible to expose something similar if this is expected behavior? Having to concatenate all of the row-level vectors into one large vector could be pretty expensive for my use case (> 20M rows in memory, 16 floats each).
To Reproduce
I will see if I can break this into minimal program that reproduces.
Expected behavior
I would expect that writing smaller batches would work
Additional context
https://github.com/lazear/sage/blob/parquet/crates/sage-cloudpath/src/parquet.rs

I just noticed I have a typo.... instead of 3i16, the definition level is 316. 😭.
This works fine, please ignore. Thanks for the great library.


Awesome script

Can the script support grid selection diagonally?
That way I could implement it on word search puzzles for an educational site we host.
Best regards
STAY SAFE! The world needs developers like you!

Heyy thanks :)
Could you elaborate the diagonal thing? You mean drawing a line and selecting all elements intersected by it?

Yes, thanks for your interest on my request.
There are a lot of wordsearch puzzle games that have that feature... but the look and smooth feel your interface has... they can't not come close to what you've accomplished so far.
some code can be run to draw the words inside a board or grid on the screen.
It would be amazing if your screen can dump under the hood... the values of the text elements so a range can be preset to detect when the whole word has been selected.
On which direction?
Any...  up --- down --- from left to right --- inverted words from right to left .... and diagonally (you have 4 axis if I am not mistaking)
You can google html5 word search games and it would give you more LIVE examples.
NOTE: Another use that I am thinking for your script would be to have a grid of user uploaded images on the screen... and that by selecting them on any direction... they can all be ... moved to another directory .... renamed or deleted from the server... Gosh that would be so neat!!!!
Best regards and STAY SAFE buddy!!!

I think that this can better be accomplished by a a custon script, using selectionjs as a helper - I want to keep it as "simple" as possible but this sounds like a rather full app to me :)


Embedding API doesn't work with text-davinci-002 or text-davicinci-003

I tested this code with text-davinci-003 and text-davinci-002:
https://github.com/TheoKanning/openai-java/blob/main/client/src/test/java/com/theokanning/openai/EmbeddingTest.java
In both cases, I get:
retrofit2.adapter.rxjava2.HttpException: HTTP 401
Has the API changed maybe? What should I use instead?

Embedding is not offered by this model as per https://platform.openai.com/docs/guides/embeddings/use-cases


vs code Debugger for Java hangs with multithreads

VSCode Version: 1.40.2
OS Version: Win 10 Pro 1903
Java Extension Pack: 0.8.1
Java Test Runner: 0.21.0
Debugger for Java: 0.23.0

Steps to Reproduce:

There is no clear way to reproduce. But the probelm happens when trying to debug (F5) a multi-threading Java application that you put break points inside it.

Does this issue occur when all extensions are disabled?: Yes, noting that the Java Extension Pack is mandatory for the test ans so I didn't disable it.

I don't think it is an extension issue, I think that I am facing the same issue described in issue 57253
I think that the same problem has been appear again
https://github.com/microsoft/vscode/issues/57253


Want to add a dino game

Project Description.
A simple dino game which looks pretty simple but provides a good insight in HTML, CSS, JS.
It's responsive.
Live Version: https://anchita15.github.io/DinoGame/

Thanks!


REST API to get resource provider operations list

Documentation indicates access to operations through CLI or PowerShell. Is there REST API equivalent to fetch the data?

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

ID: 33219707-3b9f-cc6b-541e-a806dc82622a
Version Independent ID: b4e96391-bcf9-1f30-cb44-ed339469da4f
Content: Azure Resource Manager resource provider operations
Content Source: articles/role-based-access-control/resource-provider-operations.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

@maya-cnx thank you for your query. Please check the link for the RBAC rest API reference . Please let us know in case this helps you .
Thank you.

Yes that helps. It seems the same as cli command listed here. Thank you.

Thank you for confirmation . We will now close this issue. In case you have any further queries feel free  to engage us again through the same channel.
Thank you.


Vs code can't be installed.

I have the problem of "the selected drive or UNC share does not exist or is not accessible" in the installation.

Did you pick a UNC path to install VS Code in?


APP_KEY not cached unless config:cache is run twice

Laravel Version: 8.42.1
PHP Version: 7.4.3

Description:
In one of our automated deployment processes for a pre-production environment, we have had a strange issue in the way that APP_KEY is cached. We always deploy from a clean checkout of the repository (symlinking to a persistent storage directory, but with a clean bootstrap/cache folder), and in the last steps of the process we run key:generate and cache:config.
We're noticing that if we only run cache:config once, the bootstrap/cache/config.php file will not have the APP_KEY value cached (key will just have '') - all other values are cached just fine (e.g. database passwords and so on). If we run cache:config again (without touching anything else or running any other commands), then it will have the APP_KEY value correctly cached.
Any ideas as to why we'd need to run config:cache twice? It does work this way, but it feels wrong :-).
Steps To Reproduce:

If at the end of our deployment process we run the below commands, bootstrap/cache/config.php will not have APP_KEY cached (but will have everything else cached just fine).
php artisan migrate --force --seed
php artisan cache:clear
php artisan view:clear
php artisan route:clear
php artisan key:generate
umask 0007
php artisan config:cache
php artisan up
sudo /usr/sbin/service php7.4-fpm reload

If we change it so it runs config:cache twice, it will have APP_KEY cached:
php artisan migrate --force --seed
php artisan cache:clear
php artisan view:clear
php artisan route:clear
php artisan key:generate
umask 0007
php artisan config:cache
php artisan config:cache
php artisan up
sudo /usr/sbin/service php7.4-fpm reload

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

I am having the exact same issue in production (Ubuntu EC2 instance), the first time the config:cache command is run, the APP_KEY is missing, the 2nd time it appears.
I can't replicate the issue on my dev/staging server, hovewer, even though I have it set up to resemble the production environment as as closely possible. The only major difference I can think of rn is that in production everything is run under the www-data user, while on dev/staging I run everything under a local user.


[SPARK-29861][CORE] Reduce downtime in Spark standalone HA master switch

What changes were proposed in this pull request?

Close open zookeeper connections during spark shutdown (SIGTERM)
Bump the curator version to 4.2.0
Implement a custom error policy that is tolerant to a zookeeper connection suspension.

Why are the changes needed?
As described in SPARK-29861, the recovery process of Spark (standalone) master in HA with zookeeper took about 1-2 minutes. During this time no spark master is active, which makes interaction with spark essentially impossible.
Justification for the changes:

because the spark master election does not properly restart if there are still open zookeeper connections
as a precondition for 3
to not restart spark when the connection is suspended temporarily, but only if it is really closed.

Does this PR introduce any user-facing change?
No.
How was this patch tested?

Set up spark with two masters in standalone mode and zookeeper
Kill active master
Observe that the second master takes over immediately (within a few milliseconds)

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Test build #115347 has started for PR 26598 at commit 421ddcc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20153/
Test PASSed.

Test build #115347 has finished for PR 26598 at commit 421ddcc.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/115347/
Test FAILed.

BTW, Bump the curator version to 4.2.0 is required? If not, please remove that dependency upgrade.
Yes, sadly this is part of the improvement.

The custom error policy was introduced starting curator 3.0, so we need a bump anyways. I decided to do that to the highest stable version if this needs a bump anyways.
I am looking into the failed test now.

SBT misses the class ConnectionHandlingPolicy during test runtime and therefore has one test case failing. The class is introduced in curator 3 (so should be available with the version bump).
I've had the same error locally before because of a dependency eviction and solved it introducing the exclusion for curator in the pom (at the hadoop depedency).
Now the tests succeeds for me locally in maven, but apparently not in sbt.
Is it possible the pom dependency exclusion does not get translated properly into sbt? My ivy report shows that in the phase test-internal the dependency is evicted by an old version in favor of hadoop 2.7.4.
Or do I have to add some exclusion rule equivalent in sbt to make this work there as well?


[translation] Localize app + add translations #3

Issue
https://github.com/JiggyDevs/twikkl-app/issues/3
In this PR, we introduce react-i18next to manage loading, displaying and switch between different localisation.
Focus of the PR is around the technical changes to add localisation, actual translations should be done as followup tasks on a per language basis.
Initial languages files created:

ar: Arabic
ch: Chines
en: English (default)
fr: French
hi: Hindi

For convenience, a simple language switch was added to the Profile page

I have observed that the translation files do not contain translations for each of the languages, is that intentional?

I have observed that the translation files do not contain translations for each of the languages, is that intentional?

Hi @Vcky4
As I mentioned in the PR description actual translations should be done as followup tasks on a per language basis.
This is to keep the PR small but also to ensure translations are done properly as not a lot of people can translate 5 languages.
If you want I can google translate the entries for this PR, but as a general principle I would recommend splitting adding new keys and translating to different languages as different tasks

Language files are now translated (using google translate) and language selection is persisted locally by using i18next-browser-languagedetector

that's great
I'll merge this one


Add support of priorityClassName to the operator chart

Addresses #145

@avadhut123pisal please rebase and generate examples

@avadhut123pisal please rebase and generate examples

Done


[gha] consolidate all the release/main jobs in to one ci-post-land.yml with a prepare step.

Motivation
Keeping track of which jobs will run when pushing to gh-test-* /release/main with a handful of not-standardly-named workflow files was painful.   Consolidating those files in to one with a prepare step.  Also bumps the macosx vm in gha, which is how this pr started.   Shrug.
Have you read the [Contributing Guidelines on pull requests]
Yes
Test Plan
Push to gha-test-1 and watch them run.
Related PRs
None.
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

Test here (at least the most important bits).   https://github.com/diem/diem/actions/runs/1072013812

/land ing it since I'm on call this week and deal with fallout.

:broken_heart: Test Failed - ci-test

/land

:broken_heart: Test Failed - ci-test

/land

:broken_heart: Test Failed - ci-test

/land
Third times a charm?

:broken_heart: Test Failed - ci-test

@rustielin 0/3 on helm tests

/land

:broken_heart: Test Failed - ci-test

/land

:broken_heart: Test Failed - ci-test

Sorry about the flaky test @rexhoffman . After re-basing on main, helm test won't block you any more https://github.com/diem/diem/pull/8834#event-5087446221. Tracking the issue here: https://github.com/diem/diem/issues/8836

/land good thing bors rebases automatically :D


SQLitePCLRaw.bundle_green Dependencies linux and osx?

Hi
Why would you put osx, linus and xp dependencies in SQLitePCLRaw.bundle_green.
Especially for net framework packages?
This can cause nuget/build issues... And why would we want to automatically have linux/osx for something that only runs on windows (i.e. net framework "stuff").

Because the .NET Framework TFMs are the way mono identifies itself.
I agree that the situation is unfortunate.

Hmmm. But you have the droid and iOS or linux you should install on the parent / host project, getting the correct package in.
This solution here causes sometimes build issues.

Can you be more specific?  What build issues?

@ericsink I needded time to  inveestigate.
The issues was simple.
The build server is not open to nuget.org. We have a package server,
And the linux version was not available..
I still think it is unfortunate that the SQLitePCLRaw.bundle_green depends on the linux, iosx, droid etc..
Maybe you could rethink?

"Maybe you could rethink?"
Based on the information I have, no.
The dependencies have been this way for years.  Changing the definition of this bundle would break things for many users.
And I don't see a reason why.  It's a bundle.  It's purpose is to bring in dependencies for all platforms.  For all cases where it is problematic, the solution is to not use the bundle and instead use a subset of the individual packages it depends on.


Perform Math Operations

Addition, Substraction, division

SourceCode/2023/Nov2023/Mrinali/Perform Math Operations/project.json
File: NoFile

Your project needs to have an Automation Hub URL. You can add it under 'Project Settings'.
You have a few packages (UiPath.Excel.Activities, UiPath.Mail.Activities, UiPath.Testing.Activities, UiPath.UIAutomation.Activities) in your code that aren't being used. Removing them can speed up your process.

File: Main.xaml

You're using the same names for 'Assign', 'Input Dialog' and 'Log Message' activities more than necessary. It's crucial that each activity has a unique name that gives a clear idea of what it does.
Many activities ('Assign', 'Input Dialog', 'Log Message') have default names. Try to pick names that tell other developers what each activity's role is in your program.

Score:
Code completion: 1/2 - The code is missing the Automation Hub URL, and there are unused packages.
Execution: 2/2 - No execution errors are mentioned in the review.
Annotations and comments: 0/1 - There is no evidence of use of annotations or comments in the code.


[WIP][SPARK-29887][SQL]PostgreSQL dialect: cast to smallint

What changes were proposed in this pull request?
cast to smallint behaviour in SparkSQL is made consistent with PostgreSQL when spark.sql.dialect is set to PostgreSQL.

Why are the changes needed?
SparkSQL and PostgreSQL behaviour is very different when casting to smallint.
For e.g:
select cast('abc' to smallint) in SparkSQL gives null.
select cast('abc' to smallint) in PostgreSQL gives "invalid input syntax for integer: abc"

Does this PR introduce any user-facing change?
Yes

How was this patch tested?
Manually. Test cases will be added soon.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Thank you for contribution, @iRakson . But, unfortunately, we decided to remove PostgreSQL dialect via SPARK-30125 (#26763). Sorry about that. I'll close this PR.


Update drupal/jwt to 8.x-1.0-beta2

https://www.drupal.org/project/jwt/releases/8.x-1.0-beta2

Thanks @manez @dannylamb


Can't enter letters with diacritics in RichTextBlock with Firefox

Issue Summary
With Firefox, hen I try to enter a letter with a separate diacritic in a RichTextBlock, the diacritic appears, then disappears when I type the letter, instead of being replaced by the letter with the diacritic.
If I try to do the same with Chromium, it works normally.
Steps to Reproduce

Start a new project with wagtail start mysite
Add a Streamfield with a RichTextBlock to the automatically created HomePage model

from wagtail.core import blocks
from wagtail.core.fields import StreamField
from wagtail.core.blocks import field_block
from wagtail.models import Page
from wagtail.admin.edit_handlers import FieldPanel

class HomePage(Page):
    body = StreamField(
        [("section", blocks.RichTextBlock()), ("title", field_block.CharBlock())],
        blank=True,
        use_json_field=True,
    )

    content_panels = Page.content_panels + [
        FieldPanel("body", heading="Content"),
    ]

Make the migration, migrate and follow the rest of the installation tutorial
In Firefox, open the Wagtail admin panel and edit the home page
add a "Section" field and type the diacritic (eg "^", "~" or "¨") then the letter (eg "e" or "n")
Repeat step 5 with a "Title"

Result: in the "Section" field, no character is typed (eg, if I wanted to write "même", only "mme" appears)
Expected result: the character with the diacritic should appear in the "Section" field (as it correctly does on the "Title" field)

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

Technical details
$ python --version
Python 3.10.6
$ pip show django | grep Version
Version: 4.1.3
$ pip show wagtail | grep Version

Python version: Run python --version.
Django version: Look in your requirements.txt, or run pip show django | grep Version.
Wagtail version: Look at the bottom of the Settings menu in the Wagtail admin, or run pip show wagtail | grep Version:.
Browsers versions: Firefox 107.0 (bug is present), Chromium 108 (works normally)
Not sure if relevant, but it is tested from Ubuntu 22.04 in French with the "Français (variante)" keyboard disposition.

This looks like a duplicate of #7386.

Sorry, I tried to find if the issue was present by searching "circumflex" and "diacritic", I forgot to look for "accented".

No problems at all. I do hope we get to the bottom of it.


Traefik stuck in CrashLoopBackOff

Traefik keeps crashing after a fresh install, with those logs:
time="2017-05-05T22:35:58Z" level=info msg="Traefik version v1.2.1 built on 2017-03-27_03:19:40PM"
time="2017-05-05T22:35:58Z" level=info msg="Using TOML configuration file /config/traefik.toml"
time="2017-05-05T22:35:58Z" level=info msg="Preparing server http &{Network: Address::80 TLS:<nil> Redirect:<nil> Auth:<nil> Compress:false}"
time="2017-05-05T22:35:58Z" level=info msg="Preparing server https &{Network: Address::443 TLS:0xc42035cd80 Redirect:<nil> Auth:<nil> Compress:false}"
time="2017-05-05T22:35:58Z" level=info msg="Starting server on :80"
time="2017-05-05T22:36:00Z" level=info msg="Generating ACME Account..."
time="2017-05-05T22:36:28Z" level=warning msg="Error checking new version: Get https://update.traefik.io/repos/containous/traefik/releases: dial tcp: i/o timeout"
time="2017-05-05T22:36:41Z" level=error msg="Error creating TLS config: get directory at 'https://acme-v01.api.letsencrypt.org/directory': failed to get json \"https://acme-v01.api.letsencrypt.org/directory\": Get https://acme-v01.api.letsencrypt.org/directory: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)"
time="2017-05-05T22:36:41Z" level=fatal msg="Error preparing server: get directory at 'https://acme-v01.api.letsencrypt.org/directory': failed to get json \"https://acme-v01.api.letsencrypt.org/directory\": Get https://acme-v01.api.letsencrypt.org/directory: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)"

I think it is because it takes some time for CNI networking to be setup after creating a pod and traefik times out before the networking starts working.

Turns out it was an issue with my networking, not sure there's anything to be done with the chart.


Using Vue 3 in Chrome extension bug

Version
3.0.7
Reproduction link
https://github.com/kospl/demo-vue-3-chrome-extension-issue
Steps to reproduce
Follow readme steps at https://github.com/kospl/demo-vue-3-chrome-extension-issue/tree/main#readme
What is expected?
Console showing logs (no errors).
What is actually happening?
Error in console: Error handling response: TypeError: obj.error_info.errors.filter is not a function.

I know it is a bug related not just to Vue, but to Chrome extensions, but I think as a lot of developers use Vue, it worth to be logged to be aware of this issue while migrating to Vue 3.

I know it is a bug related not just to Vue, but to Chrome extensions

Can you provide more details? You say this is a general bug in chrome.storage? Then we can't really do anything about this ...

@LinusBorg this was working with Vue 2 without an issue. While updating to Vue 3 it happened to not save array data correctly. I am not aware for exact reason, this might be too deep to investigate for me personally.
From provided demo it's easy to scope the issue to Vue / Proxy and Chrome storage API, but I am not sure which one of these three is a reason for data saved incorrectly, probably the combination Chrome APIs + Proxy is just wrong, or Chrome APIs + Vue's Proxy implementation.
I am still trying to find workaround, as deep copying method before saving seems has no effect.

toRaw() can give you the non-proxy original data, you should probably save that to storage instead.
chrome.storage.local.set({ error_info: Vue.toRaw(this.error_info) }, function(){

Might be worth documenting.


tcp_nodelay fix

In the latest version, the only way to turn ON tcp_nodelay (reduces jitter for real-time games), is to replace websocketserver.java with the following (it will then ALWAYS BE ON):
(You can thank me quietly, Im only posting this because others here have helped with their posted info- giving back) Would be great if this was made a public option for the lib
WebSocketServer.java.zip

Hello @clickstan,
could you please open a pull request for your commit?
Greetings
marci4

I'm actually interested in learning GIT more, so i'll do it!

possible duplicates #339 (internal for me)

added with #462

https://github.com/TooTallNate/Java-WebSocket/wiki/Enable-TCP_NODELAY


Trib to Missinka models as potentially accessible but no crossing present

@lucy-schick spotted this one

linear_feature_id = 165107895

That segment is coded in the FWA as being downstream of PSCIS 125191 - because blue_line_key 359016386 is a side channel / distributary of blue_line_key 359502337.
To avoid this issue, future models could build the network directly from the stream geometries (rather than using watershed codes), but that would be a big change - we have to live with this for now.
There is something slightly weird going on though - that took me longer than it should to diagnose because clicking on that feature in QGIS shows and empty value for the column barriers_pscis_dnstr. The correct data is present in the view (see below) - but is not showing up in QGIS. It shows up fine if querying streams_vw.
bcfishpass=# select barriers_pscis_dnstr from bcfishpass.streams_vw where linear_feature_id = 165107895;
 barriers_pscis_dnstr
----------------------
 125191
(1 row)

bcfishpass=# select barriers_pscis_dnstr from bcfishpass.streams_bt_vw where linear_feature_id = 165107895;
 barriers_pscis_dnstr
----------------------
 125191


The brand icons don't work

If you select an icon from the brand collection (for exemple "adobe"), the icon is displayed as a square.

Thanks for reporting! Should be fixed now (e75678b68eabbbcfc5992dfe350ee9660a1f27ff).

It's working now. Thank you for the quick fix ! 👍


[#956] django rendering monkey patching

Monkey patch django.shortcuts.render and TemplateResponse to have a django.render span in the sentry performance view.
One of the features suggested in #956 that @untitaker saw as useful and not requiring any django configuration update.

Hum, the error seems to be a pip install issue. It may have impacted the coverage detection. How to re run Travis for confirmation?
(I have 2 of my 70 lines which are not tested, the use of default behavior if integration is not enabled which is took from elsewhere in the same file and not tested there either.)

@untitaker happy new year, I'm allowing myself to ping you so you can tell me if this is ok or how I should improve this.
Note: it only monkey patch main django rendering entries. it should work with any django template engine.
Note 2: it only provide the auto rendering span, not the template tag as I'm not sure I can do it in the current philosophy (no settings) while django discourage to add these keywords

here the issue where django says they don't want lib to add template tags to builtins https://code.djangoproject.com/ticket/17085
I see how we can monkey-patch it as loadable without being and installed django app but if we do it and a user load it from a template, this template would break if you remove sentry's Django integration at render time.

@untitaker happy new year, I'm allowing myself to ping you so you can tell me if this is ok or how I should improve this.
Note: it only monkey patch main django rendering entries. it should work with any django template engine.
Note 2: it only provide the auto rendering span, not the template tag as I'm not sure I can do it in the current philosophy (no settings) while django discourage to add these keywords

here the issue where django says they don't want lib to add template tags to builtins https://code.djangoproject.com/ticket/17085
I see how we can monkey-patch it as loadable without being and installed django app but if we do it and a user load it from a template, this template would break if you remove sentry's Django integration at render time.

@Christophe31 You raise good points about the stability of the injected tag and I think I agree
I reviewed the PR
with regard to CI, we switched to GitHub actions in master. I'm going to close and reopen the PR to trigger a fresh build

@Christophe31 You raise good points about the stability of the injected tag and I think I agree
I reviewed the PR
with regard to CI, we switched to GitHub actions in master. I'm going to close and reopen the PR to trigger a fresh build

should I pull master to have the best tests configurations?

should I pull master to have the best tests configurations?

@untitaker I disabled dj1.6 monkey patch on django.shortcut.render to pass the tests as I had an import error (it required loaded settings)

@untitaker I disabled dj1.6 monkey patch on django.shortcut.render to pass the tests as I had an import error (it required loaded settings)

Excellent, thanks!

Excellent, thanks!


How to store object in store ? Seems hmset is not supporting.

__

hmset is specific to redis, so it's not supported by node-cache-manager because it needs to be generic for a variety of cache stores.


RuntimeError: Exporting the operator argsort to ONNX opset version 11 is not supported.

Describe the bug
When running  pytorch2onnx.py script against a Mask-RCNN model, I get the error about argsort operator. Could MMDet code shift to using sort operator, as this :
https://github.com/pytorch/pytorch/issues/33412
issue says?
Reproduction

What command or script did you run?

 python pytorch2onnx.py /my/model/config.py /my/model/weights.pth

Did you make any modifications on the code or config? Did you understand what you have modified?

I encountered this error with unmodified mmdet code. Then, I changed all occurences of .argsort in mmdet like this:
original:
x = y.argsort(...)

to:
_, x = y.sort(...)

What dataset did you use?

Custom dataset in COCO format. This problem is about an already successfully-trained model, which I'd like to convert to ONNX.
Environment

Please run python mmdet/utils/collect_env.py to collect necessary environment information and paste it here.

Python: 3.6.9 (default, Jul 17 2020, 12:50:27) [GCC 8.4.0]
CUDA available: True
GPU 0: GeForce RTX 2080 Ti
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 10.1, V10.1.243
GCC: gcc (Ubuntu 7.4.0-1ubuntu1~18.04.1) 7.4.0
PyTorch: 1.8.0+cu101
PyTorch compiling details: PyTorch built with:
  - GCC 7.3
  - C++ Version: 201402
  - Intel(R) Math Kernel Library Version 2020.0.0 Product Build 20191122 for Intel(R) 64 architecture applications
  - Intel(R) MKL-DNN v1.7.0 (Git Hash 7aed236906b1f7a05c0917e5257a1af05e9ff683)
  - OpenMP 201511 (a.k.a. OpenMP 4.5)
  - NNPACK is enabled
  - CPU capability usage: AVX2
  - CUDA Runtime 10.1
  - NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_70,code=sm_70
  - CuDNN 7.6.3
  - Magma 2.5.2
  - Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=10.1, CUDNN_VERSION=7.6.3, CXX_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/c++, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_KINETO -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=1.8.0, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON,

TorchVision: 0.9.0+cu101
OpenCV: 4.4.0
MMCV: 1.2.5
MMCV Compiler: GCC 7.4
MMCV CUDA Compiler: 10.1
MMDetection: 2.10.0+b9e2af5

You may add addition that may be helpful for locating the problem, such as

PyTorch installed with pip.
Error traceback
If applicable, paste the error trackback here.
Traceback (most recent call last):
  File "pytorch2onnx.py", line 246, in <module>
    cfg_options=args.cfg_options)
  File "pytorch2onnx.py", line 58, in pytorch2onnx
    opset_version=opset_version)
  File "/usr/local/lib/python3.6/dist-packages/torch/onnx/__init__.py", line 276, in export
    custom_opsets, enable_onnx_checker, use_external_data_format)
  File "/usr/local/lib/python3.6/dist-packages/torch/onnx/utils.py", line 94, in export
    use_external_data_format=use_external_data_format)
  File "/usr/local/lib/python3.6/dist-packages/torch/onnx/utils.py", line 698, in _export
    dynamic_axes=dynamic_axes)
  File "/usr/local/lib/python3.6/dist-packages/torch/onnx/utils.py", line 465, in _model_to_graph
    module=module)
  File "/usr/local/lib/python3.6/dist-packages/torch/onnx/utils.py", line 206, in _optimize_graph
    graph = torch._C._jit_pass_onnx(graph, operator_export_type)
  File "/usr/local/lib/python3.6/dist-packages/torch/onnx/__init__.py", line 309, in _run_symbolic_function
    return utils._run_symbolic_function(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/torch/onnx/utils.py", line 990, in _run_symbolic_function
    symbolic_fn = _find_symbolic_in_registry(domain, op_name, opset_version, operator_export_type)
  File "/usr/local/lib/python3.6/dist-packages/torch/onnx/utils.py", line 947, in _find_symbolic_in_registry
    return sym_registry.get_registered_op(op_name, domain, opset_version)
  File "/usr/local/lib/python3.6/dist-packages/torch/onnx/symbolic_registry.py", line 112, in get_registered_op
    raise RuntimeError(msg)
RuntimeError: Exporting the operator argsort to ONNX opset version 11 is not supported. Please feel free to request support or submit a pull request on PyTorch GitHub.

Up to now, MASKRCNN is not supported for pytorch2onnx.

Thank you!


Add Hotjar tracking

What is hotjar tracking?

It's a website analytics tool - good for heatmapping, and UX analytics
https://www.hotjar.com/


FutureWarning…

I keep getting this:
/usr/local/lib/python3.10/dist-packages/diffusers/configuration_utils.py:134: FutureWarning: Accessing config attribute use_karras_sigmas directly via 'DPMSolverSDEScheduler' object attribute is deprecated. Please access 'use_karras_sigmas' over 'DPMSolverSDEScheduler's config object instead, e.g. 'scheduler.config.use_karras_sigmas'.
deprecate("direct config name access", "1.0.0", deprecation_message, standard_warn=False)
It doesn't seem to actually cause any problems, just thought I'd let you know it keeps showing up.

yes that kind of deprecated warning always appears, can be safe to ignore, thanks for posting that


Refactor order price calculation.

The main idea of the PR is to decouple order level discounts applying logic from tax calculation and reuse it for flat rates and tax exemption scenarios. Handling discounts for TAX_APP strategy is kept unchanged.
It fixes price calculation for tax exemption scenario, where order level discounts were not reflected in shipping and order lines prices.
Discount types
We can distinguish discounts by:

type

manual

percentage
fixed

voucher

entire order (name can be misleading, as it is applied to subtotal only, not shipping)
shipping
specific product

promotions (are included in channel listing variant prices)

level

order level discounts

manual (applied to both subtotal and shipping price)
entire order voucher
shipping voucher

line level discounts

promotions
specific product voucher
manual (applied to unit price, not total line price; subject to change: https://github.com/saleor/saleor/issues/14879)

This PR assumes all below statements are valid.
Order line prices

base_unit_price_amount includes line level discounts, but excludes order level discounts; is a price needed for tax applications
undiscounted_base_unit_price_amount = undiscounted_unit_price_net_amount (not sure here!!!)
total_price_net_amount includes all discounts
total_price_net_amount = quantity * base_unit_price_amount + propagated order level discount
undiscounted_total_price_net_amount = quantity * undiscounted_base_unit_price_amount
unit_price_net_amount = total_price_net_amount / quantity
undiscounted_unit_price_net_amount = undiscounted_total_price_net_amount / quantity
unit_discount_amount includes line level discounts, but excludes order level discounts
unit_discount_amount = undiscounted_base_unit_price_amount - base_unit_price_amount
unit_discount_type - since multiple discounts can be applied to the line, this should be deprecated (we can use OrderLineDiscount objects instead)
unit_discount_value - since multiple discounts can be applied to the line, this should be deprecated (we can use OrderLineDiscount objects instead)

Order prices

undiscounted_total_net_amount = sum(line.base_unit_price * line.quantity for line in order.lines.all()) + base_shipping_price
undiscounted_total_net_amount includes line level discounts, but excludes order level discounts
total_net_amount = sum(line.total_price_net_amount for line in order.lines.all()) + shipping_price_net_amount
total_net_amount is a final price including all discounts
base_shipping_price_amount includes shipping voucher discount
shipping_price_net_amount includes all discounts (shipping voucher discount + order level manual discount)

AC
Order level manual discounts and entire order vouchers are correctly handled (meaning all the above assumptions are met) for:

flat rate tax startegy
tax exemption scenario

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations

Docs

[ ] Link to documentation:

Pull Request Checklist

[ ] Privileged queries and mutations are either absent or guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migrations are either absent or optimized for zero downtime
[ ] The changes are covered by test cases
[ ] All new fields/inputs/mutations have proper labels added (ADDED_IN_X, PREVIEW_FEATURE, etc.)
[ ] All migrations have proper dependencies
[ ] All indexes are added concurrently in migrations
[ ] All RunSql and RunPython migrations have revert option defined

The assumptions listed below Order line prices chapter are valid only when provided price in Saleor are in .net (you're operating there on net value. If you will have Saleor prices provided in gross all statements listed there will be valid for gross.

undiscounted_base_unit_price_amount = undiscounted_unit_price_net_amount (not sure here!!!)

I might be wrong, but This could not true. Base is calculated by Saleor based on the prices in the database. The undiscounted can be a value calculated by plugin. You could have here a difference like 0.01 (but I am not totally sure, how it exactly looks like after providing the flat rates feature)

unit_price_net_amount = total_price_net_amount / quantity

You have total_price_net_amount = quantity * base_unit_price_amount + propagated order level discount
Assuming, that your total_price_net_amount is 9,05 (assuming after applying the discount) and qt:2, if you make a  total_price_net_amount / quantity you will get 4.525, which after rounding gives you 4.52 or 4.53. So using the unit price to calculate the total you will get 9.04 or 9.06 which is different than previous one.
So in the case of calculating the prices we should avoid flows like this one:
unit_price -> unit_price * quantity (total) -> total/quantity(unit_price)

It seems like we are not calculating the discounts for SPECIFIC_PRODUCT and APPLY_ONCE_PER_ORDER. Am I right or I'm missing something?

You are right. Also SHIPPING vouchers are note handled at the moment. I am working on it in separate PR.


Fix Style/Documentation false-positive offence #1905.

Fixes #1905.

Is there a test that can exercise the changed code?

Overall the changes look good. Update the changelog, fix the typos, squash the commits, refer to the issue in the commit message and capitalise the commit message.

Looks good, but unfortunately you'll have to rebase on top of the current master.

@RGBD You'll have to rebase again.

That was fast! Thanks!


[Google Drive] Optimize file previews cache

Description
Optimizes file previews cache -

Reuses already generated file preview instead of trying to generate a new one every time the same file is accessed.
On init, now the extension checks if the maximum number of file previews (500) is exceeded or not. If yes, it removes the least accessed file previews to free up the disk space.

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi @vishaltelangre 👋
Thanks for another significant update 🔥
I tried to run the command Clear File Previews Cache and got this error:

I think I would move DB_FILE_PATH to environment.supportPath instead, since tmp might be cleared unintentionally.

@pernielsentikaer

I tried to run the command Clear File Previews Cache and got this error

Fixed it in https://github.com/raycast/extensions/pull/1546/commits/d532c9ba959ef0cec413157214810204d9b9628e.

I think I would move DB_FILE_PATH to environment.supportPath instead, since tmp might be cleared unintentionally.

DB_FILE_PATH is situated in the user's home directory and not in /tmp directory. I am okay with moving it to the extension's support directory (assuming Raycast will clean it up while uninstalling the extension).
I would continue to keep the temporary file previews in /tmp though since they're not super essential (and are generated on-demand if don't exist). I understand that /tmp gets cleaned up on the  computer restart and it's an advantage for storing temporary file previews in there.

Thanks for the clarification, I just wanted to make sure that it was intentionally 😄


Suggestion

Is it possible if u can make a player join more then 1 party! unless this is allready added and im just to lazy to try it out :P

I have intentionally made it impossible for that to happen. There's no good way for the user to know/specify what party or global chat they want to speak in.


chore: increase timeout for selftests

commit 7d3973cb954873b8a056bd3af968ce81ab176463
chore: increase timeout for selftests

All selftests are now running with a timeout of 10 seconds, except for
attachgenericfd, which has now its own run.sh with a timeout of
15 seconds.

Selftests were failing from time to time due to timeout https://github.com/aquasecurity/libbpfgo/actions/runs/5726626218/job/15517444077#step:4:1438.


Google will no longer allow OAuth requests to Google in embedded browsers

Hi Nic,
What do you think about the following link? https://developers.googleblog.com/2016/08/modernizing-oauth-interactions-in-native-apps.html?m=1
Regards, Nicholls

Support for Google Oauth will more than likely be dropped in ng-cordova-oauth.  The goal of this library was to use the consistent Oauth spec to prevent having to mess around with X number of native provider SDKs.
It is a lot easier to work with a formal spec and support X number of providers than it is to work with X number of very different SDKs.
Best,

No need to drop support – Google still supports 100% standards-based OAuth for native apps, no native SDK needed. We're just dropping support for embedded browsers (i.e. web-view) as a way to do OAuth, since it's not as user friendly or secure.
You can set InAppBrowser to use the system browser like so:
cordova.InAppBrowser.open('https://accounts.google.com/…', '_system');

You'll also need to register your app for the custom URI scheme callback (on iOS, macOS, Android, and Universal Windows Platform).  If you're using a platform where this callback is not possible, then it's OK to continue to use embedded browsers for OAuth to Google on those platforms.
As a UX upgrade on the system browser, it's advisable to use in-app browser-tabs when available (this is especially important on iOS, as it's frowned upon to open mobile Safari for sign-in).  We released a library recently to help with that: https://github.com/google/cordova-plugin-browsertab. See the sample code in the README that includes fallback handling (so on compatible versions of iOS and Android it would open the browser tab, otherwise it will revert to the system browser).
Not all OAuth providers support this improved method of OAuth for Native Apps, but more and more are doing so, so you can upgrade each as they support it.
I understand this is a bit of work to switch over, but note that there is a large user benefit if you do: your users won't have to sign-in to Google again to use your app!  Users who are already signed-in to Google will be able to tap a single button and get in!  Users who are not signed-in will have to sign-in to Google for the first app they use, but then won't have to again for subsequent apps.  This should increase the conversation rate of your sign-in flow dramatically (our user research shows a lot of people abandon the sign-in completely when they have to sign-in from scratch, as is always the case with web-view).

I am pretty certain the system browser will not work because the event handlers don't work the same in Apache Cordova.  If the host application cannot anticipate the end of the login flow, extract the token, and close the browser, then it will not work.
I have no plans to support each and every native oauth SDK for every provider.  If the InAppBrowser no longer functions for a particular provider, that provider will be removed.  My reasoning for this is that only a small percentage have Apache Cordova plugins and I have no intention of making plugins for the providers that don't.
@WilliamDenniss if you work for Google (sounds like you do), I encourage you to submit a PR to this project fixing Google support.  This is a heavily used library and it will prove how accommodating Google is to the platforms they disrupt.
Also, unless I'm missing something, I don't see how removing web-view support makes oauth any more secure.  Implicit grant types were meant to be done in client facing interfaces.  I can see from a friendliness perspective, but I don't really agree with the secure part that Google is selling (unless I'm missing something).

@WilliamDenniss : How would you close that system browser and get back to the app?
@nraboy is right, the support required for OAuth (loadstart for the redirect url) for system browser does not seem to be available for that scenario:
addEventListener

Browser Quirks
loadstart and loaderror events are not being fired.

I validated the InAppBrowser browser plugin (we were actually using) and no events were fired using cordova-plugin-inappbrowser 1.5.0 on an iPhone 6 / iOS 10

In general terms, the OAuth provider redirects the user (in the system browser) to a custom URI scheme claimed by the app, which opens the app. The Authorization Response URL is provided to the app by the OS when it opens.  The app can then process the authorization response URI as before.
I have personally tested this flow using custom URI schemes on iOS, Android and UWP (with native sample apps, not cordova apps). I believe there should be a way in Cordova to receive the redirect URIs when the app is opened in this manner.
I found the instructions to handle custom URI schemes for Codova on iOS here. This solution looks appropriate for receiving Android intents.

Google support was removed from the library in https://github.com/nraboy/ng-cordova-oauth/commit/555b5caafbe0b7a552c6dc418e3353f6615f24da.  The preferred way to use Google now is to use the native Apache Cordova plugin.
Best,

Google! :-1:

Users who are not signed-in will have to sign-in to Google for the first app they use, but then won't have to again for subsequent apps.

@WilliamDenniss What if I don't want the user's Google credentials saved on the device for whatever reason? Is there still a way around it?

Try this plugin for Google Auth instead
https://github.com/EddyVerbruggen/cordova-plugin-googleplus

once I've moved over to another system (still unclear what "the native Apache Cordova plugin" is), will my users who've signed in via Google be able to login via the new system or will I need to migrate their accounts ?

I have a Cordova iOS app whose resources (js, CSS, etc...) reside on a remote server and I use the Firebase 2.x SDK (authWithOAuthPopup) to authenticate once, then it automatically caches the token and prevents future oauth popups. I want my app (an email app) to remain logged in to the Google account that the user originally auth'd, even if they log in to a different google account somewhere else.  Will this change 1) prevent me from using Firebase's auth popup after April, and 2) log my app's user out of the their gmail account if they subsequently log in to different google account in another app?

A workaround to this problem is to add this to your config.xml. It prevents Google from detecting the request is from a mobile app. It worked for me.
  <preference name="OverrideUserAgent" value="Mozilla/5.0 Google" />

@slayer3600 it indeed works but it stops Ionic adding platform specific classes on CSS, and also ionic.Platform.isIOS() and ionic.Platform.isAndroid() stop returning false, and other few annomalies - so be careful :P

@nraboy with the solution of @slayer3600 you can add google again, what do you think?

Google has dropped support.  I don't think it is appropriate to add a hack to get it working.  As @ChrisEelmaa mentioned, there are other problems it brings up.
If you want Google support, I suggest using a native SDK alternative.

@slayer3600 thanks! that did the trick for me, it might not be the best solution but it buys you time to implement something more stable.
I don't use iconic but my app is made with angular, and angular material as a css framework and everything is working correctly with this trick.

@ChrisEelmaa tried that, but that plugin doesn't seem to work. UserAgent.set("Mozilla/5.0 Google") results in unrecognized selector sent to instance
So my app is broken currently. Angry mails incoming..

@FPSE  yes I know that's why I said that plugin itself is outdated.
Also one warning is that OverrideUserAgent is not shown as supported for iOS on the latest docs: https://cordova.apache.org/docs/en/latest/config_ref/index.html
You will see that AppendUserAgent is shown iOs/Android both, but not the Override one.
That said, I use Satellizer to do the authentication, and it used InAppBrowser as well for login flow, fortunately, it was relatively easy to replace the inAppBrowser pop ups with native ones, as the code is quite good.
I don't know if this will help, but maybe ng-cordova-oauth has the same way - is there a way to replace the google popups with native popup?
You can see more @ https://github.com/sahat/satellizer/issues/1029 where I show the code (I use cordova-plugin-googleplus)

@WilliamDenniss Using system browser combined with a custom URL scheme works a treat. Thanks!

Guys, check my example using Firebase and cordova-plugin-googleplus https://gist.github.com/jdnichollsc/17bae6581d405321937fe433410172c9
Also you can check the comparative with the old way using ng-cordova-oauth
Regards, Nicholls

Does anyone have a fix incase we have Keycloak in between? We are using an ionic based app which does google auth via keycloak.

How to handle an OAuth situation like trakt.tv has?
They provide a login screen, which has Email/Pass, Facebook, Twitter and Google sign-in capabilities. I'm using ng-cordova-oauth and everything works fine, except when a user tries to sign-in with Google from the trakt.tv OAuth screen (which is loaded in the WebView).
More info: http://docs.trakt.apiary.io/#reference/authentication-oauth
It appears I would have to load the trakt.tv OAuth screen (which has the google sign in button) from the system Chrome/Safari app but as @jmcrthrs pointed out, Apple doesn't like that.
So what options do we have left?
My app users are complaining...

@m-vdv https://github.com/EddyVerbruggen/cordova-plugin-safariviewcontroller
I'm going to use this plugin for iOS (and stick with system browser for Android)

@m-vdv, you've described our situation perfectly.  We use Auth0 as our identity provider.  They host a login page that contains multiple provider options (Google/Facebook/Email+Pass).  Historically, we'd use inappbrowser to navigate to the Auth0 login page.  Today, we use cordova-plugin-safariviewcontroller (for both Android and iOS) to hit that Auth0 login page using PKCE.  This works for most use-cases but there's some gotchas:

Neither Facebook or Google allow you to login without cookies being enabled.  Because cordova-plugin-safariviewcontroller relies on the default system browser, users can disable cookies, leading to a poor login experience.
cordova-plugin-safariviewcontroller relies on Chrome custom tabs for Android.  If your default browser wasn't Chrome, it wouldn't work.  I actually submitted a PR to search for all installed browsers so we could warn the user if Chrome wasn't installed.  If you're using an older version of the plugin, you may want to update.
To provide a better user experience, we programatically check for cookie support to warn the user they should enable cookies before logging in.  Our app temporarily hosts a web server at localhost that immediately redirects back to our app (using url scheming) to report back if cookies are enabled on the system browser using a small amount of javascript.  We've found that some antivirus programs such as Total AV do not like custom scheme redirects, IE "UNKNOWN_URL_SCHEME"s.

We still have users reporting occasional login issues as Android fragmentation pops up to bite us.  Hoping

If anyone would be willing to share a working example (or tutorial) of the system browser method (for Android) or the 'cordova-plugin-safariviewcontroller' method proposed by @setheen that would be great!

It's been 5 days now, I feel like committing suicide or something.  I am facing many  issues. Everytime I solve one, new one comes up. Its like never ending loop. What is the matter with this Google and Apple companies. This is the first time I am developing an App and with ionic.

I have ionic 3 App for Android, iOS, Windows
I am using Cordova inAppBrowser for Google oAuth. Same thing I have to use _system target otherwise it wouldn't work.
Now I am using deeplink plugin for custom URL scheme. Unfortunately that is also not straight forward way. I had to create a sample html page to redirect to myapp://example.com/pagename because apparently google is too fancy to accept oauth redirect with that form.
Finally my app is opening after redirect but guess what, it opens in the system browser I just opened for google oauth.
Checkout following image.

Please someone help me out. If you have any working sample code of this please give me. I feel no hope anymore.


Add curtain model to description

This pull request adds the curtain mode to the discover section.

Thank you.


delegate: gate estimated_row_count on SHOW TABLES behind cluster setting

PR: https://github.com/cockroachdb/cockroach/pull/60282
From release notes:

Added the sql.show_tables.estimated_row_count.enabled cluster setting, which defaults to true. If false, estimated_row_count will not display on SHOW TABLES which improves performance. [#60282][#60282] {% comment %}doc{% endcomment %}

Fixed by https://github.com/cockroachdb/docs/pull/9944.
(see follow-up PR for backport: https://github.com/cockroachdb/docs/pull/10229)


OID4VCI: Align RFC001 with latest approved version 13

OpenID for Verifiable Credential Issuance Draft 13 is approved.
https://openid.net/specs/openid-4-verifiable-credential-issuance-1_0.html

[ ] Check all fields in all messages
[ ] Draft 13 seems like using credential_configuration_ids over credentials in EWC RFC001

Are we planning to switch on a specific date?

Are we planning to switch on a specific date?

We can decide this jointly. My idea was to get it up in June as this doesn't seem so urgent. Do share your thoughts.
We need to agree also on a process how we introduce the changes. For now, the first release v1.0 holds and our target for next release (2.0) is June. I am planning to have a bi-weekly follow up for new RFCs that we need to introduce, mainly PID / LPID issuance, wallet attestation and payment scenarios.

June sounds good

@lalc I think regular meetings in which we are discussing the RFC sounds like a good idea.
For the release of a new version of the RFC I would suggest to consider a date that does not interfere with the piloting and pilot preparation processes.

+1 for June

Based on this conversation, we will make a new release on 01-June.  Please confirm. @endimion @georgepadayatti @ntsbs @mat-work @andreasabr

Hi everybody,
I think its good to move forward. Nevertheless, I think if we decide to move to draft v13, we have to spend time reviewing the changes made in the new version and also if our RFC complies with this version. This will take some effort, and I doubt that it can be completed on time (at least from our side).
On the other hand, we agreed a long time ago that we should not do RFC updates right before the piloting, in order to give the partners time to work with a stable version and to focus on the piloting.
Finally, the benefits of moving to v13 is not clear why that is considered before the piloting since we agreed months ago that SD will not be part of the piloting. The suggestion from the old conversations was to make simple things right instead of overcomplicating things. Piloting should be an incremental process, starting with a simple and minimal setup now and advancing its complexity for the final large-scale piloting.

I think its good to move forward. Nevertheless, I think if we decide to move to draft v13, we have to spend time reviewing the changes made in the new version and also if our RFC complies with this version. This will take some effort, and I doubt that it can be completed on time (at least from our side).

Please suggest a suitable time before the end of June to agree on moving to draft 13. Draft 13 is the approved spec, and we don't want to be lagging behind.
@endimion @georgepadayatti @ntsbs @mat-work @andreasabr

Hi @lalc, at SICPA, we are aiming to release Draft 13  between the first and second weeks of June.

Mid of June (14th June) should be fine for us

Since we are currently focusing on the piloting, we would only start working on reviewing and adopting the RFC in a bit. Therefore, I would aim at the beginning of July to be able to release the updated RFC.
Keep in mind, this discussion is about when to release the updated version of the RFC and not when an implementation is ready.

Here is what I propose based on this discussion:

Schedule the release of version 2 of the EWC RFCs, adopting draft 13 of OID4VCI, for June 28.
Compliance with ITB standards is the responsibility of wallet providers. The release of the EWC RFCs should proceed independently of the wallet providers' compliance plans.

Thank you @lalc. I agree, and we can proceed with your suggestion.

Hi, the link for reference [1] in RFC001 is pointing to draft-14 of openid4vci. Could you please update it to point to https://openid.net/specs/openid-4-verifiable-credential-issuance-1_0-13.html instead?

Hi, the link for reference [1] in RFC001 is pointing to draft-14 of openid4vci. Could you please update it to point to https://openid.net/specs/openid-4-verifiable-credential-issuance-1_0-13.html instead?

Noted. Will fix the same in the PR. Will use the implementor draft 1 (ID1) URL which in turn points to draft 13. https://openid.net/specs/openid-4-verifiable-credential-issuance-1_0-ID1.html

Hi, the link for reference [1] in RFC001 is pointing to draft-14 of openid4vci. Could you please update it to point to https://openid.net/specs/openid-4-verifiable-credential-issuance-1_0-13.html instead?

Annoying when standards orgs do not follow basic naming rules.☺

I think that in the issuer metadata example (chapter 3.4 Discover response) cryptographic_suites_supported should be credential_signing_alg_values_supported.

This is now fixed as part of PR #63

Closing this issue as part of PR #63 merge


Add a prerequisite section for disks used by cStor

Cover the prerequisites for disks like:

Should they be unmounted or cleared of any older FS?
Should  they be partitioned or not?
Can LVM disks be used?
Can external SAN mounted disks be used?

(Q) Should they be unmounted or cleared of any older FS?
Yes, the disks should be unmounted and better it should be cleared of any file system, though there is an option -f that can be given which wipe out the file system on the disk. We have to check if control plane is using -f or not.
(Q) Should  they be partitioned or not?
No. Partition is not supported yet.
(Q) Can LVM disks be used?
No. Currently we are not supporting LVMs. LVMs will be excluded by path filter.
(Q) Can external SAN mounted disks be used?
Yes. External SAN disks will be detected by NDM and can be used by cStor.


New release

Could we get a new release on Hackage please?  Commits such as 0eec6193df0a36b94709e3d4751648023edd262e are needed for Alex to work on GHC HEAD.
Unfortunately we cannot just use source-repository-package in our cabal.project to include this repository, because of the manual bootstrapping shenanigans that are needed to prepare an sdist for Alex.

Can you try out https://hackage.haskell.org/package/alex-3.2.7/candidate/ and let me know how it goes?

I can build that one with GHC HEAD.  I can also build it with GHC 8.10.7, so it's probably good!

Thanks!