-
Notifications
You must be signed in to change notification settings - Fork 251
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
Uses redirectexit and fixes #3798 #3799
Uses redirectexit and fixes #3798 #3799
Conversation
Signed-off-by: Colin Schoen <cschoen@berkeley.edu>
Signed-off-by: Colin Schoen <cschoen@berkeley.edu>
Fixing indentation, one moment... |
Signed-off-by: Colin Schoen <cschoen@berkeley.edu>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
redirectexit()
calls obExit()
, and the both of them combined are laced with arsenic and eat the image alive.
Working locally. What goes wrong when using them? |
iDunno
|
Image larger than threshold is working. I'll try the other tomorrow |
Deleting the image and refreshing the page is also working when checking out this PR locally. Not able to replicate any issues. @live627 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Derp. I was testing against the wrong version.
No worries. Thanks for reviewing. |
Use redirectexit instead of manually changing headers. Fixes #3798
@jdarwood007