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.

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

fix 403 response when hitting the VA notify email endpoint for appoint #96311

Closed
holdenhinkle opened this issue Nov 1, 2024 · 0 comments · Fixed by department-of-veterans-affairs/vets-website#32845
Assignees
Labels
accredited-representation-management-team Accredited Representation Management team frontend mvp Initial version of thing

Comments

@holdenhinkle
Copy link
Collaborator

Slack convo for context:

Josh Fike
20 minutes ago
I'm working on testing the VA Notify email and I'm getting a 403 Forbidden response from the server. The controller for that endpoint looks like most of the other ones in the representation_management module with skip_before_action :authenticate present. The only difference I see in the chrome inspector network tab is that the request before this one for PDF generation sends along a Cookie and a X-CSRF-Token. The request for the next steps email endpoint doesn't have either of those.
Does missing those two fields or maybe the Cookie specifically seem like a cause for the 403 Forbidden?
Is it straightforward to add those fields to the next steps email request?
2 replies

Holden Hinkle
17 minutes ago
Yes, I think the X-CSRF-Token (and possibly the cookie) is the issue - it allows the api to sort of authenticate the request.

Josh Fike
16 minutes ago
I'm suspicious of the cookie because I'm not getting a CSRF error and I know those are usually specifically called out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accredited-representation-management-team Accredited Representation Management team frontend mvp Initial version of thing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants