-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
uri: Add form-urlencoded support to body_format #37188
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dagwieers
force-pushed
the
uri-formencoded
branch
from
March 8, 2018 14:27
fc11015
to
bbccbe3
Compare
ansibot
added
core_review
In order to be merged, this PR must follow the core review workflow.
feature
This issue/PR relates to a feature request.
module
This issue/PR relates to a module.
needs_triage
Needs a first human triage before being processed.
net_tools
Net-tools category
support:core
This issue/PR relates to code supported by the Ansible Engineering Team.
test
This PR relates to tests.
labels
Mar 8, 2018
dagwieers
force-pushed
the
uri-formencoded
branch
from
March 8, 2018 15:26
bbccbe3
to
b490681
Compare
ansibot
added
stale_ci
This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested.
and removed
stale_ci
This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested.
labels
Mar 17, 2018
@romeotheriault Can we merge this ? |
sivel
reviewed
Mar 22, 2018
dagwieers
force-pushed
the
uri-formencoded
branch
4 times, most recently
from
March 22, 2018 17:35
c5d71bf
to
f1c71ed
Compare
ansibot
added
needs_revision
This PR fails CI tests or a maintainer has requested a review/revision of the PR.
and removed
core_review
In order to be merged, this PR must follow the core review workflow.
labels
Mar 22, 2018
dagwieers
force-pushed
the
uri-formencoded
branch
from
March 22, 2018 20:26
f1c71ed
to
c0ccfb7
Compare
This comment has been minimized.
This comment has been minimized.
mattclay
added
the
ci_verified
Changes made in this PR are causing tests to fail.
label
Mar 24, 2018
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
ansibot
removed
the
ci_verified
Changes made in this PR are causing tests to fail.
label
Mar 26, 2018
ansibot
added
the
needs_revision
This PR fails CI tests or a maintainer has requested a review/revision of the PR.
label
Apr 12, 2018
This PR adds form-urlencoded support so the user does not need to take care of correctly encode input and have the same convenience as using JSON. This fixes ansible#37182
dagwieers
force-pushed
the
uri-formencoded
branch
from
April 16, 2018 20:02
cb1270e
to
9f45edd
Compare
ansibot
removed
the
stale_ci
This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested.
label
Apr 16, 2018
ansibot
added
the
stale_ci
This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested.
label
Apr 25, 2018
ansibot
added
stale_review
Updates were made after the last review and the last review is more than 7 days old.
and removed
stale_ci
This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested.
labels
May 3, 2018
ansibot
added
the
stale_ci
This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested.
label
May 10, 2018
No longer my problem
dagwieers
force-pushed
the
uri-formencoded
branch
from
May 10, 2018 16:19
9d982df
to
230074b
Compare
ansibot
removed
stale_ci
This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested.
stale_review
Updates were made after the last review and the last review is more than 7 days old.
labels
May 10, 2018
Ready for re-review. |
dagwieers
force-pushed
the
uri-formencoded
branch
from
May 10, 2018 16:32
f475c54
to
934cfff
Compare
@sivel Can we get this merged before v2.6 is branched ? |
sivel
approved these changes
May 17, 2018
rebuild_merge |
ansibot
added
core_review
In order to be merged, this PR must follow the core review workflow.
and removed
needs_revision
This PR fails CI tests or a maintainer has requested a review/revision of the PR.
labels
May 17, 2018
achinthagunasekara
pushed a commit
to achinthagunasekara/ansible
that referenced
this pull request
May 23, 2018
* uri: Add form-urlencoded support to body_format This PR adds form-urlencoded support so the user does not need to take care of correctly encode input and have the same convenience as using JSON. This fixes ansible#37182 * Various fixes * Undo documentation improvements No longer my problem * Fix the remaining review comments
jacum
pushed a commit
to jacum/ansible
that referenced
this pull request
Jun 26, 2018
* uri: Add form-urlencoded support to body_format This PR adds form-urlencoded support so the user does not need to take care of correctly encode input and have the same convenience as using JSON. This fixes ansible#37182 * Various fixes * Undo documentation improvements No longer my problem * Fix the remaining review comments
ilicmilan
pushed a commit
to ilicmilan/ansible
that referenced
this pull request
Nov 7, 2018
* uri: Add form-urlencoded support to body_format This PR adds form-urlencoded support so the user does not need to take care of correctly encode input and have the same convenience as using JSON. This fixes ansible#37182 * Various fixes * Undo documentation improvements No longer my problem * Fix the remaining review comments
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
core_review
In order to be merged, this PR must follow the core review workflow.
feature
This issue/PR relates to a feature request.
module
This issue/PR relates to a module.
net_tools
Net-tools category
support:core
This issue/PR relates to code supported by the Ansible Engineering Team.
test
This PR relates to tests.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
SUMMARY
This PR adds form-urlencoded support so the user does not need to take
care of correctly encoded input and have the same convenience as using
JSON.
Compatible with Requests.
This fixes #37182
ISSUE TYPE
COMPONENT NAME
uri
ANSIBLE VERSION
v2.5