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.

Sign up for GitHub

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

Jump to bottom

stream=True should only be used for list_replicas POST calls #1249

Closed
bari12 opened this issue Jun 14, 2018 · 1 comment
Closed

stream=True should only be used for list_replicas POST calls #1249

bari12 opened this issue Jun 14, 2018 · 1 comment
Assignees
@bari12
Labels
Clients enhancement Priority: High
Milestone
1.16.4-clients

Comments

@bari12
Copy link
Member

bari12 commented Jun 14, 2018

Motivation

At the moment it is used for ALL POST calls, but it should only be done for the list_replica one where it is needed.
@bari12
Copy link
Member Author

bari12 commented Jun 14, 2018

@mlassnig @nikmagini @TomasJavurek can one of you take over this ticket? It would actually be nice if we can get this into 1.17.0 (on Monday). If nobody has time I can take it as well.

@bari12 bari12 assigned bari12 and unassigned mlassnig, nikmagini and TomasJavurek Jun 15, 2018
bari12 added a commit to bari12/rucio that referenced this issue Jun 15, 2018
@bari12
Clients: POST requests should not be submitted in streaming mode; Fix r…
b072902 
…ucio#1249
This was referenced Jun 15, 2018
Merged
Merged
bari12 added a commit that referenced this issue Jun 15, 2018
@bari12
Merge pull request #1256 from bari12/patch-1249-stream_True_should_no…
6baae33 
…t_be_used_for_POST

Clients: POST requests should not be submitted in streaming mode; Fix #1249
bari12 added a commit that referenced this issue Jun 15, 2018
@bari12
Merge pull request #1255 from bari12/patch-1249-stream_True_should_no…
58a9a0a 
…t_be_used_for_POST

Clients: POST requests should not be submitted in streaming mode; Fix #1249
@bari12 bari12 added this to the 1.16.4-clients milestone Jun 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bari12 bari12

Labels
Clients enhancement Priority: High
Projects
None yet
Milestone
1.16.4-clients
Development

No branches or pull requests
4 participants
@mlassnig @bari12 @nikmagini @TomasJavurek