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

Sending mails in usermin fails with v19.00 #842

Closed
andyhirsch opened this issue Oct 4, 2017 · 6 comments
Closed

Sending mails in usermin fails with v19.00 #842

andyhirsch opened this issue Oct 4, 2017 · 6 comments

Comments

@andyhirsch
Copy link

With Authentic Theme 19.00 sending mail in usermin produces following error:
Expecting form-data encoding but got normal encoding

If I revert to 18.49-9 everything's fine again.

@iliajie
Copy link
Collaborator

iliajie commented Oct 8, 2017

I see the cause of the issue.

Thanks for reporting.

@alejandromalo
Copy link

I'm getting same "Expecting form-data encoding but got normal encoding" error message when trying to restore backups from different sources (local, FTP or S3) and everything works fine if I change theme.

@iliajie
Copy link
Collaborator

iliajie commented Oct 10, 2017

Those forms have enctype set, with get method, which is wrong, I think. Changing it to post should work.

With the next patch, I'll make sure that all of those forms will always be submitted using post method.

@iliajie
Copy link
Collaborator

iliajie commented Oct 10, 2017

Actually, for the restore page it's not the reason I mentioned above.

@iliajie
Copy link
Collaborator

iliajie commented Oct 10, 2017

Both of the issues are going to be fixed on the next commit.

I'm surprised that it's actually possible to fix the issue with progressive output and multipart/form-data.

iliajie pushed a commit that referenced this issue Oct 10, 2017
@iliajie
Copy link
Collaborator

iliajie commented Oct 10, 2017

It's fixed in latest git version.

Thank you for reporting, appreciate that a lot.

@iliajie iliajie closed this as completed Oct 10, 2017
iliajie pushed a commit that referenced this issue Nov 3, 2017
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

No branches or pull requests

3 participants