@nejch Just wanted to get the ball rolling on this. Not sure how much should be in the PR.

Some unit tests are going to be wanted. Not sure about functional tests.

How about placing the function in the backend?

How about placing the function in the backend?

Since I don't know much about the backend, can you explain why? I'm doing it like all the otherhttp_* methods which currently exist.

How about placing the function in the backend?

Since I don't know much about the backend, can you explain why? I'm doing it like all the otherhttp_* methods which currently exist.

Moving the http_* functions to the backend has already started. It will save a future work of moving this one as well and there will be no need to change the function signature.

Moving the http_* functions to the backend has already started. It will save a future work of moving this one as well and there will be no need to change the function signature.

Sorry. I haven't looked into the back-end code. Can you explain how that will work? I don't really see much of anything in the back-end. I don't see any of thehttp_* methods there. EDIT: I seehttp_request

If all the otherhttp_* methods need to be moved then I'm not sure how this one extra function is going to add much to the work required.

Moving the http_* functions to the backend has already started. It will save a future work of moving this one as well and there will be no need to change the function signature.

Sorry. I haven't looked into the back-end code. Can you explain how that will work? I don't really see much of anything in the back-end. I don't see any of thehttp_* methods there. EDIT: I seehttp_request

If all the otherhttp_* methods need to be moved then I'm not sure how this one extra function is going to add much to the work required.

You might want to take a look at#2435

Codecov Report

Merging#2471 (b27f4ee) intomain (0867564) willincrease coverage by0.00%.
The diff coverage is100.00%.

📣 This organization is not using Codecov’sGitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories.Learn more

@@           Coverage Diff           @@##             main    #2471   +/-   ##=======================================  Coverage   96.18%   96.19%           =======================================  Files          86       86             Lines        5635     5644    +9     =======================================+ Hits         5420     5429    +9  Misses        215      215

Flags with carried forward coverage won't be shown.Click here to find out more.

@JohnVillalovos, I think this could be afeat as people can already use it for unimplemented endpoints after merge.

Moving the http_* functions to the backend has already started. It will save a future work of moving this one as well and there will be no need to change the function signature.

Sorry. I haven't looked into the back-end code. Can you explain how that will work? I don't really see much of anything in the back-end. I don't see any of thehttp_* methods there. EDIT: I seehttp_request
If all the otherhttp_* methods need to be moved then I'm not sure how this one extra function is going to add much to the work required.

You might want to take a look at#2435

@lmilbaum in any case we will have these user-facing helper methods on the client, preferably backend-agnostic (https://python-gitlab.readthedocs.io/en/stable/api-levels.html#lower-level-api-http-methods). They help users work around endpoints we haven't implemented in the library.

I hope if we get this right onlyhttp_request and the associated typing will need a significant backend-specific implementation and the rest can wrap around it like we do now.

@JohnVillalovos, I think this could be afeat as people can already use it for unimplemented endpoints after merge.

Moving the http_* functions to the backend has already started. It will save a future work of moving this one as well and there will be no need to change the function signature.

Sorry. I haven't looked into the back-end code. Can you explain how that will work? I don't really see much of anything in the back-end. I don't see any of thehttp_* methods there. EDIT: I seehttp_request
If all the otherhttp_* methods need to be moved then I'm not sure how this one extra function is going to add much to the work required.

You might want to take a look at#2435

@lmilbaum in any case we will have these user-facing helper methods on the client, preferably backend-agnostic (https://python-gitlab.readthedocs.io/en/stable/api-levels.html#lower-level-api-http-methods). They help users work around endpoints we haven't implemented in the library.

I hope if we get this right onlyhttp_request and the associated typing will need a significant backend-specific implementation and the rest can wrap around it like we do now.

I am not following what you are trying to say. In any case, maybe switching the return type in a similar way I've done in the PR#2435.

I am not following what you are trying to say. In any case, maybe switching the return type in a similar way I've done in the PR#2435.

@lmilbaum since this PR is a separate concern, I think we can do that after, I can also rebase your PR withhttp_patch to avoid more rebase issues on your end. I think we can go ahead with this one just to unblock users who want to update protected branches in#2390.