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

Exclude `file` field `value` from `FormElement.form_values`. #199

merged 1 commit into from Jul 21, 2016


None yet
2 participants
Copy link

tdivis commented Jul 20, 2016

Similar to submit, image and reset, browsers don't send file field values in the POST when form is submitted. FormElement.form_values method already correctly excluded submit, image and reset fields, now it also excludes the file fields.

Tomas Divis
Exclude `file` field `value` from `FormElement.form_values`.
Similar to `submit`, `image` and `reset`, browsers don't send `file` field values in the POST when form is submitted. `FormElement.form_values` method already correctly excluded `submit`, `image` and `reset` fields, now it also excludes the `file` fields.

This comment has been minimized.

Copy link

scoder commented Jul 21, 2016

Ok, seems reasonable. Thanks.

@scoder scoder merged commit b41ee9f into lxml:master Jul 21, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment