Skip to content

Includes csrf token more directly.#58

Merged
hbradio merged 3 commits intodevelopmentfrom
bugfix/No_csrf_token_in_prod
Apr 29, 2017
Merged

Includes csrf token more directly.#58
hbradio merged 3 commits intodevelopmentfrom
bugfix/No_csrf_token_in_prod

Conversation

@hbradio
Copy link
Copy Markdown
Collaborator

@hbradio hbradio commented Apr 28, 2017

When running in production, the csrf token was not available in the cookie. So, this includes it directly in the DOM as shown here.

When running in production, the csrf token was not available in the cookie.
@hbradio hbradio added this to the Release 0.4 milestone Apr 28, 2017
@hbradio hbradio requested a review from cabarnes April 28, 2017 12:20
@hbradio
Copy link
Copy Markdown
Collaborator Author

hbradio commented Apr 28, 2017

Whoops. Need to do the prospector workaround. Will do after work.

Copy link
Copy Markdown
Member

@cabarnes cabarnes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well that was a lot easier

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling b8e639f on bugfix/No_csrf_token_in_prod into 99aa45b on development.

@hbradio hbradio merged commit f49362d into development Apr 29, 2017
@hbradio hbradio deleted the bugfix/No_csrf_token_in_prod branch April 29, 2017 01:16
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.

3 participants