Skip to content
This repository has been archived by the owner on May 31, 2024. It is now read-only.

Fix status filtering #28

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

jon-rtr
Copy link
Contributor

@jon-rtr jon-rtr commented Jan 23, 2020

From rundeck/rundeck#1923:

I've traced the issue down to the Content-Type header that messes up the whole request (It's not needed for a GET anyway, was there from a previous POST). The output is still a XML, I'll open a different issue for that

There are a few posts saying the same thing: GET requests specifying a
Content-Type header causes the URL parameters to be ignored.

Ignore JetBrains IDE-generated files.
Run the black formatter over the code.
As of 2020 Jan 01, official support for Python 2 has ended (https://www.python.org/doc/sunset-python-2/, https://python3statement.org/).

* Remove `from __future__ import absolute_import` statement.
* Remove `from __future__ import unicode_literals` statement.
* Remove `from __future__ import print_function` statement.
* Remove `from urlparse import urljoin` statement, and retain the Python
3 equivalent.
* Switch to using format strings.
* BUGFIX: Per https://docs.rundeck.com/docs/api/rundeck-api.html#bulk-delete-executions, the url in the bulk_delete_executions function should look like `{}/executions/delete`, not `{}/executions/{}/delete`.
From rundeck/rundeck#1923:
> I've traced the issue down to the Content-Type header that messes up the whole request (It's not needed for a GET anyway, was there from a previous POST). The output is still a XML, I'll open a different issue for that

There are a few posts saying the same thing: GET requests specifying a
Content-Type header causes the URL parameters to be ignored.
@jon-rtr jon-rtr requested a review from pschmitt January 23, 2020 02:35
@pschmitt pschmitt added the needs-rebase PR needs to be rebase against the current master label Apr 20, 2021
Copy link
Owner

@pschmitt pschmitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rebase.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs-rebase PR needs to be rebase against the current master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants