@pablobm commented on this pull request.
> @@ -233,9 +233,16 @@
get "/forgot-password.html", :to => redirect(:path =>
"/user/forgot-password")
# omniauth
- get "/auth/failure" => "users#auth_failure"
- match "/auth/:provider/callback" => "users#auth_success", :via => [:get,
:post], :as => :auth_success
- match "/auth/:provider" => "users#auth", :via => [:post, :patch], :as =>
:auth
+ scope "/auth", :as => :auth do
+ get "/failure" => "users#auth_failure"
+
+ scope ":provider" do
+ match "/callback" => "users#auth_success", :via => [:get, :post], :as =>
:success
+ match "" => "users#auth", :via => [:post, :patch]
+
+ resource :delete, :only => [:show, :create], :module => "accounts",
:controller => "auth_deletions"
Is it possible to tell Facebook to put it in the parameter? I guess it could
work. I just learned about Base64URL for the first time and I see it should be
ok there (no slash `/` or anything that could confuse the router).
But if not, I think `resource :deletions`, with a comment to explain the
situation, should be ok.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7093#discussion_r3494984807
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/7093/review/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
