We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Our payment processor uses a 500 error response to indicate that a customer is already enrolled in a group.
500
This is not an actual problem for Benefits, so we have code that handles the response and returns a success to the user.
The error handling is brittle, relying to some degree on the returned error message. This message has now changed, thus our handling is broken.
Steps to reproduce the behavior:
Attempting to re-enroll with the same card should not cause an error for the end user.
New error message from the logs looks like:
Customer xxxx already exists in group yyyy
The old one looked more like:
Duplicate customer xxxxx...
The text was updated successfully, but these errors were encountered:
Sentry issue: BENEFITS-48
Sorry, something went wrong.
thekaveman
Successfully merging a pull request may close this issue.
Our payment processor uses a
500
error response to indicate that a customer is already enrolled in a group.This is not an actual problem for Benefits, so we have code that handles the response and returns a success to the user.
The error handling is brittle, relying to some degree on the returned error message. This message has now changed, thus our handling is broken.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Attempting to re-enroll with the same card should not cause an error for the end user.
Additional context
New error message from the logs looks like:
The old one looked more like:
The text was updated successfully, but these errors were encountered: