Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rack 3 #670

Merged
merged 4 commits into from
Aug 1, 2023
Merged

Rack 3 #670

merged 4 commits into from
Aug 1, 2023

Conversation

jnunemaker
Copy link
Collaborator

A few tweaks to help with rack 3 support. There might be more but I can't actually update until rails does and its currently locked to rack 2. Once rails git supports rack 3 I can update this PR and see if tests pass.

xref #669

@jnunemaker jnunemaker self-assigned this Oct 6, 2022
@n-rodriguez
Copy link

Once rails git supports rack 3 I can update this PR and see if tests pass.

rails/rails#45741

@ericauv
Copy link

ericauv commented Jan 20, 2023

When is this expected to get merged?

@bkeepers
Copy link
Collaborator

Whenever Rack 3 support lands in Rails. See rails/rails#46594

@ericauv
Copy link

ericauv commented Jan 20, 2023

Potentially related, let me know if this should be a separate issue -- is it possible to relax the rack-protection version to include up to 3.0.5 ?

@rgarver
Copy link

rgarver commented Jun 14, 2023

It looks like Rails edge is not supporting Rack 3: rails/rails#48460

@bvogel
Copy link

bvogel commented Jul 24, 2023

the mentioned issue on rails was solved/merged, anything still in the way to get rack 3 support?

* v1.0: (170 commits)
  Release 0.28.3
  Update changelog
  Remove sync interval from cloud config sync
  Avoid any possibility of sync writing back to remote
  Release v0.28.2
  UI: fix path to bundled assets when mapped to a prefix
  Prepare for 0.28.1
  Update changelog
  Improve performance of `color_name` function in log subscriber
  Update Changelog.md
  remove remote source from CSP.
  Bundle bootstrap, jquery and poppler.
  Add mirroring example
  Use new method of making logs bold
  Add a key_prefix option to the Redis adapter
  Fix typo in comment
  Update Changelog.md
  Add threadsafe: option to memory adapter
  Ensure memory adapter unlocked on fork
  Get rid of shotgun
  ...
@bkeepers bkeepers changed the base branch from main to v1.0 August 1, 2023 14:29
@bkeepers bkeepers marked this pull request as ready for review August 1, 2023 14:29
@bkeepers bkeepers merged commit 65726db into v1.0 Aug 1, 2023
50 checks passed
@bkeepers bkeepers deleted the rack-3 branch August 1, 2023 14:29
@bvogel
Copy link

bvogel commented Aug 2, 2023

@bkeepers YEAY!!! Thank you. 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants