Upgrade to Rails 7.1 #9333
Merged
Upgrade to Rails 7.1 #9333
Conversation
This file contains 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
mitchellhenke
force-pushed
the
mitchellhenke/rails-7-1
branch
from
3d21e54
to
1fc4509
Compare
mitchellhenke
commented
Oct 10, 2023
| attempts: 2, | ||
| ) | ||
| retry_on RedisRateLimiter::LimitError, | ||
| wait: :exponentially_longer, | ||
| wait: :polynomially_longer, |
There was a problem hiding this comment.
This is not a change in behavior, just a rename to be more accurate: rails/rails#49292
mitchellhenke
commented
Oct 10, 2023
| @@ -55,7 +55,6 @@ class Application < Rails::Application | |||
|
|
|||
| config.load_defaults '7.0' | |||
| config.active_record.belongs_to_required_by_default = false | |||
| config.active_record.legacy_connection_handling = false | |||
There was a problem hiding this comment.
this option was deprecated and is now removed: https://github.com/rails/rails/blob/358d3d3164bef399b19b089f5fed8664a2c3d39e/guides/source/7_1_release_notes.md#removals-5
mitchellhenke
commented
Oct 10, 2023
| @@ -1,6 +1,6 @@ | |||
| Rails.application.configure do | |||
| config.cache_classes = true | |||
| config.cache_store = :redis_cache_store, { url: IdentityConfig.store.redis_url } | |||
| config.cache_store = :redis_cache_store, { url: IdentityConfig.store.redis_url, pool: false } | |||
There was a problem hiding this comment.
These have pooled connections by default now: https://guides.rubyonrails.org/upgrading_ruby_on_rails.html#memcachestore-and-rediscachestore-now-use-connection-pooling-by-default, this keeps the same behavior
There was a problem hiding this comment.
We probably want pooled connections, right?
There was a problem hiding this comment.
I'm not sure, I'd rather address that separately.
mitchellhenke
commented
Oct 10, 2023
| @@ -100,6 +100,8 @@ ignore_unused: | |||
| - 'errors.messages.*' | |||
| - 'simple_form.*' | |||
| - 'time.*' | |||
| - 'datetime.dotiw.words_connector' | |||
There was a problem hiding this comment.
These are not used by us directly, but rather by dotiw
spec/presenters/two_factor_authentication/selection_presenter_spec.rb
Outdated
Show resolved
Hide resolved
spec/presenters/two_factor_authentication/set_up_selection_presenter_spec.rb
Outdated
Show resolved
Hide resolved
zachmargolis
approved these changes
Oct 10, 2023
mitchellhenke
force-pushed
the
mitchellhenke/rails-7-1
branch
from
8012a9c
to
b2122af
Compare
mitchellhenke
force-pushed
the
mitchellhenke/rails-7-1
branch
from
b2122af
to
a632e70
Compare
mitchellhenke
added a commit
that referenced
this pull request
Oct 11, 2023
This reverts commit f9a0cd0.
mitchellhenke
added a commit
that referenced
this pull request
Oct 11, 2023
Merged
Merged
jmdembe
added a commit
that referenced
this pull request
Oct 12, 2023
* LG-11082 Add Conditional Text To FullAddressSearch Component (#9331) * Add conditional text to view * add new tests * Add period to display text * package version increase from 3.1.0 to 3.1.1 * fix linter errors * changelog: Upcoming feature, USPS Full Address Search, Added conditional logic to display/hide text on the Find a participating Post Office view that will display in Help Center only * Integrate personal key feature specs into end_to_end_idv feature specs (#9336) Since feature specs run slowly, it's better to check assertions as part of a single longer spec rather than restart identity verification over and over. This removes several long-running feature specs from the test suite. [skip changelog] * Update specs to initialize session as HashWithIndifferentAccess (#9347) changelog: Internal, Automated Testing, Improve accuracy of session stubbing in tests * Change `<b>` tags to `<strong>` for better accessibility and code consistency (#9349) * Change `<b>` tags to `<strong>` for better accessibility and code consistency changelog: User-facing Improvements, Accessibility, Use strong html tag instead of b for emphasis * Enable RSpec/LeakyConstantDeclaration rubocop (#9348) * Enable RSpec/LeakyConstantDeclaration rubocop changelog: Internal, Source code, Enable RSpec rubocop * Use let instead of defining new class --------- Co-authored-by: Mitchell Henke <mitchell.henke@gsa.gov> * Sync TypeScript-ESLint versions (#9352) changelog: Internal, Dependencies, Update dependencies to their latest versions * LG-10037: display warning banner on gpo welcome back page if number of gpo letter requests exceeded (#9303) * display warning banner on gpo welcome back page if gpo letter requests are spammed changelog: User-Facing Improvements, Identity Verification, display warning banner if user has sent max letter requests within a time window * handle if user has no gpo confirmatio codes * Update app/views/idv/by_mail/enter_code/index.html.erb Co-authored-by: Andrew Duthie <andrew.duthie@gsa.gov> * tests for alert banner for spammed gpo requesets * happy linting * fix extra space in alert_spam_warning_html i18n * happy linting * lintfix i18n * lint line too long * js tag removal from alert gpo spam banner spec * integrate warning alert banner for spammed gpo letter requests into existing tests * refactor test for gpo spam warning banner * happy linting * create before action to remove test order dependency * happy linting * define gpo_verification_enabled in review app * define gpo_verification_enabled in review app --------- Co-authored-by: Andrew Duthie <andrew.duthie@gsa.gov> * Upgrade to Rails 7.1 (#9333) * fix otp missing translations * rails 7.1 changelog: Internal, Dependencies, Upgrade to Rails 7.1 * fix untranslated webauthn verification * LG-10837: Add New Piv Cac Logging for login visited (#9294) * changelog: Internal Fixes, Authentication LG-10837: Piv Cac Logging fixes * changelog: Internal, Authentication, Add Login visited for pivcac/change logging names to be uniform * uniform spec test * fix naming convention for piv cac * update rspec * add previous name * Add lint check for reasonable asset bundle sizes (#9353) * Add lint check for reasonable asset bundle sizes changelog: Internal, Automated Testing, Add test for reasonable asset bundle size * TEMPORARY: Revert "Fix JavaScript dead code elimination (#9217)" This reverts commit 0fcc3a7. * Revert "TEMPORARY: Revert "Fix JavaScript dead code elimination (#9217)"" This reverts commit af166f2. * Update changelog script to reflect non-security Dependabot usage (#9354) changelog: Internal, Changelog, Update changelog script to reflect non-security Dependabot usage * Revert "Upgrade to Rails 7.1 (#9333)" (#9356) This reverts commit f9a0cd0. * LG-10812 | Report on all-time user count (#9350) changelog: Internal, Reporting, Monthly report includes all-time user count * Reorganize combined invoice report for easier manual runs (#9358) changelog: Internal, Reporting, Reorganize combined-invoice-supplement-report * Exclude 'IRS Attempt API: Event metadata' events from log results (#9360) changelog: Internal, Data Requests, Exclude 'IRS Attempt API: Event metadata' events from log results * Remove Guardfile, guard dependencies (#9364) changelog: Internal, Dependencies, Remove unused testing dependencies * LG-11066 Do not redirect users at the phone step unless they are phone and address rate limited (#9345) Users are being rate limited and encounting the phone error screen even if they can still verify by mail. This commit changes the rate limit logic to allow users to proceed to the phone step if they can still verify their phone or complete verification by mail. A side-effect of this change is a bug is fixed where the following situation would exist: 1. A user proofed by mail after exhausting phone attempts 2. The user goes to GPO entry and chooses to cancel and start over 3. The user is redirected to the welcome step to start over 4. The welcome step before action observes the user is phone rate limited and sends the user to the phone errors controller 5. The phone errors controller has a before action to confirm the user has completed the phone errors step; the user has not since in this session so they are redirected to the welcome step 6. Steps 4 and 5 complete until there are too many redirects [skip changelog] Co-authored-by: Sonia Connolly <sonia.connolly@gsa.gov> --------- Co-authored-by: gina-yamada <125507397+gina-yamada@users.noreply.github.com> Co-authored-by: Sonia Connolly <sonia.connolly@gsa.gov> Co-authored-by: Andrew Duthie <andrew.duthie@gsa.gov> Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com> Co-authored-by: Mitchell Henke <mitchell.henke@gsa.gov> Co-authored-by: Amir Reavis-Bey <amir.reavis-bey@gsa.gov> Co-authored-by: Malick Diarra <malick.diarra@gsa.gov> Co-authored-by: Matt Wagner <mattwagner@navapbc.com> Co-authored-by: Jonathan Hooper <jonathan.hooper@gsa.gov>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
馃洜 Summary of changes
A few oddities in here to work out, most notably rails/rails#47105 which breaks tests in a few places where we add errors that do not have error messages (though the messages are not displayed as far as I can tell)