-
Notifications
You must be signed in to change notification settings - Fork 202
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
Add firebase auth support to endpoints sample. #96
Conversation
Fixes #32 |
logging and language are failing, appengine/endpoints passes:
|
Awesome! Taking a look... |
|
||
before do | ||
content_type :json | ||
end | ||
|
||
use Rack::Cors do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice find with Rack::Cors 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aja pointed me to: https://til.hashrocket.com/posts/4d7f12b213-rails-5-api-and-cors
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
1 newline to add 😄
get "/auth/info/firebase" do | ||
auth_info | ||
end | ||
options "/auth/info/firebase" do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: can you add a newline above options "..." do
so the handlers are separated by 1 newline?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated
4dd850c
to
054a436
Compare
Updated |
I couldn't find any example clients or docs for testing firebase auth in the Python or Go examples, so going with this until we have more docs on https://cloud.google.com/endpoints/docs/authenticating-users