Sitelet https://github.com/omniauth/omniauth/pull/1108
Skip to content

Add inspect to the raise user_info in readme - #1108

Open
ombr wants to merge 1 commit into
omniauth:masterfrom
ombr:patch-1
Open

ombr wants to merge 1 commit into
omniauth:masterfrom
ombr:patch-1

Conversation

@ombr

@ombr ombr commented Mar 9, 2023 •

Copy link
Copy Markdown

I think we should have inspect when we raise the error in the readme.

Without inspect we are getting the following error:

TypeError (exception class/object expected):

app/controllers/sessions_controller.rb:8:in `raise'

Whereas with inspect we are seeing the object:

#<OmniAuth::AuthHash credentials=#<OmniAuth::AuthHash expires=false token="XXX"> extra=#<OmniAuth::AuthHash bot_info=

Without inspect we are getting the following error:

```
TypeError (exception class/object expected):

app/controllers/sessions_controller.rb:8:in `raise'
```

Whereas with inspect we are seeing the object:

```
#<OmniAuth::AuthHash credentials=#<OmniAuth::AuthHash expires=false token="XXX"> extra=#<OmniAuth::AuthHash bot_info=
```
@ombr ombr changed the title Add inspect to raise Add inspect to the raise user_info in readme Mar 9, 2023
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.

1 participant