Skip to content

Conversation

@PGijsbers
Copy link
Collaborator

No description provided.

@mfeurer
Copy link
Collaborator

mfeurer commented Oct 6, 2022

Hey, just checking, why is "flow_exists" a POST request? It is stateless and doesn't post anything on the server, so it should be a get request?

@PGijsbers
Copy link
Collaborator Author

From @joaquinvanschoren:

I'm seeing that both are implemented, but the GET version is deprecated

and

I'm guessing that the issue is caused by problematic characters in the URL, or maybe the URL is cut off, so we now use POST instead

which might make sense given the external version or name content

@PGijsbers PGijsbers merged commit 6da0aac into develop Oct 10, 2022
@PGijsbers PGijsbers deleted the flow_get_to_post branch October 10, 2022 08:42
PGijsbers added a commit to Mirkazemi/openml-python that referenced this pull request Feb 23, 2023
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

Successfully merging this pull request may close these issues.

3 participants