chore: upgrade to Node 22.22.2 and fix yarn build#131
Merged
matiasperrone-exo merged 1 commit intoMay 11, 2026
Conversation
- Bump NODE_VERSION 20.20.2 → 22.22.2 in Dockerfile and .nvmrc - Replace node-sass@8 (EOL, no Node 22 binary) with sass@^1.77 (dart-sass) - Bump webpack@^5.94, webpack-cli@^5.1.4 (resolves OpenSSL-3 legacy hash issue) - Bump copy-webpack-plugin@^11 for webpack 5.94+ compatibility - Add util:false to webpack.common.js resolve.fallback (webpack 5.106 is stricter than 5.70 about unresolved Node built-ins; jsdom/tough-cookie transitively require util which should not be browser-polyfilled) - Drop unused devDeps: clean-webpack-plugin, html-webpack-plugin, optimize-css-assets-webpack-plugin, path polyfill, webpack-node-externals - Regenerate yarn.lock
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
📘 OpenAPI / Swagger preview ➡️ https://OpenStackweb.github.io/openstackid/openapi/pr-131/ This page is automatically updated on each push to this PR. |
matiasperrone-exo
approved these changes
May 11, 2026
matiasperrone-exo
merged commit 34a6926
into
feat/migrate-from-google-recaptcha-to-cloudflare-turnstile
6 checks passed
smarcet
added a commit
that referenced
this pull request
May 27, 2026
* feat: Migrate from Google reCAPTCHA to Cloudflare Turnstile * chore: Change to vendor recommended implementation * feat: Update login attempts retrieval to use user data if available * chore: Add PR's requested changes and Unit tests - Validation failure when cf-turnstile-response is missing - Login flow behaviour before/after CAPTCHA threshold - Rendering of Turnstile on the thresholded login screen - Auth form submission when Turnstile is expired or not solved * chore: upgrade to Node 22.22.2 and fix yarn build (#131) - Bump NODE_VERSION 20.20.2 → 22.22.2 in Dockerfile and .nvmrc - Replace node-sass@8 (EOL, no Node 22 binary) with sass@^1.77 (dart-sass) - Bump webpack@^5.94, webpack-cli@^5.1.4 (resolves OpenSSL-3 legacy hash issue) - Bump copy-webpack-plugin@^11 for webpack 5.94+ compatibility - Add util:false to webpack.common.js resolve.fallback (webpack 5.106 is stricter than 5.70 about unresolved Node built-ins; jsdom/tough-cookie transitively require util which should not be browser-polyfilled) - Drop unused devDeps: clean-webpack-plugin, html-webpack-plugin, optimize-css-assets-webpack-plugin, path polyfill, webpack-node-externals - Regenerate yarn.lock * chore: Add PR's requested changes * chore: Add PR's requested changes * chore: Fix composer.lock update issue * feat: Add remote IP address to Turnstile verification request * fix: add timeout and scope retry to connection errors in TurnstileClient - Add ->timeout(5) to bound max wall time to ~15s during Cloudflare outages - Restrict retry to ConnectionException only so success:false responses (valid HTTP 200) do not trigger wasteful retries --------- Co-authored-by: sebastian marcet <smarcet@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ref: https://app.clickup.com/t/86b8xw6ue