Skip to content

Instantly share code, notes, and snippets.

@jaredbeck
Created February 7, 2018 22:26
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save jaredbeck/2a7a05a268235fbfecf95a46f1886797 to your computer and use it in GitHub Desktop.
Save jaredbeck/2a7a05a268235fbfecf95a46f1886797 to your computer and use it in GitHub Desktop.
notes for params discussion
# 512
Authentication .. no longer possible without using Strong Parameters
https://github.com/binarylogic/authlogic/issues/512
Closed by #558, which deprecates passing AC::Parameters to authlogic. Going
forward, perform the necessary permit, and pass a plain Hash to authlogic.
# 558
Deprecate ActionController::Parameters
https://github.com/binarylogic/authlogic/pull/558
Merged into 3-stable
Produces a deprecation warning if AC::Parameters is passed
Intended to be released in 3.7.0, but that hasn't happened yet.
# 576
Remove special treatment for AC::Parameters
https://github.com/binarylogic/authlogic/pull/576
Adds dep on actionpack
# 577
https://github.com/binarylogic/authlogic/pull/577
jaredbeck wants to merge .. into master
Supports AC::Parameters (calls to_h) but raises an ArgumentError if
required keys are missing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment