Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[dacs rails applications] login is broken / need to add a header #5403

Closed
8 tasks done
sandbergja opened this issue Oct 2, 2024 · 0 comments · Fixed by #5404
Closed
8 tasks done

[dacs rails applications] login is broken / need to add a header #5403

sandbergja opened this issue Oct 2, 2024 · 0 comments · Fixed by #5404
Assignees
@christinach
Labels
bug

Comments

@sandbergja
Copy link
Member

sandbergja commented Oct 2, 2024 •
edited by christinach
Loading

Expected behavior

We can log in to repec and lockers-staging

Actual behavior

We get this:

HTTP Origin header (https://repec-prod.princeton.edu) didn't match request.base_url (http://repec-prod.princeton.edu)

Steps to replicate

  1. Go to https://repec-prod.princeton.edu/
  2. Try to Login

Impact of this bug

Users can't log in

Implementation notes, if any

Proactively apply the fix from #5402 to

  • bibdata
  • lib jobs
  • allsearch api
  • lockers (currently broken on staging, confirm that this fixes it)
  • dss
  • approvals
  • repec (currently broken on prod, confrim that this fixes it)
  • geac
@sandbergja sandbergja added the bug label Oct 2, 2024
@sandbergja sandbergja changed the title [repec and lockers-staging] login is broken / need to add a header [dacs rails applications] login is broken / need to add a header Oct 2, 2024
@christinach christinach self-assigned this Oct 2, 2024
christinach added a commit that referenced this issue Oct 2, 2024
@christinach
[#5403] Add proxy_set_header X-Forwarded-Proto https; in the dacs ngi…
ddfa1ae 
…nx conf files

to fix `HTTP Origin header (https://repec-prod.princeton.edu) didn't match request.base_url (http://repec-prod.princeton.edu)`
@christinach christinach mentioned this issue Oct 2, 2024
Merged
sandbergja pushed a commit that referenced this issue Oct 2, 2024
@christinach
[#5403] Add proxy_set_header X-Forwarded-Proto https; in the dacs ngi…
958811b 
…nx conf files (#5404)

to fix `HTTP Origin header (https://repec-prod.princeton.edu) didn't match request.base_url (http://repec-prod.princeton.edu)`
@sandbergja sandbergja mentioned this issue Oct 2, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@christinach christinach

Labels
bug
Projects
None yet
Milestone
No milestone
2 participants
@sandbergja @christinach