pyDataverse icon indicating copy to clipboard operation
pyDataverse copied to clipboard

Authentication by API token, not parameter passing

Open skasberger opened this issue 4 years ago • 1 comments

Authentication is done by passing an API token to the class instantiation, not by passing the parameter auth to the API methods.

Remove auth and establish set of API token if passed.

Prepare

  • [ ] Get all effected functions
    • [ ] get_request() ??
    • [ ] post_request() ??
    • [ ] delete_request() ??
    • [ ] put_request() ??

Actions

  • [ ] Develop tests
  • [ ] Remove auth from API methods
  • [ ] Update request methods, so they use the member-variable api_token if set and pass it to the request function
    • [ ] get_request() ??
    • [ ] post_request() ??
    • [ ] delete_request() ??
    • [ ] put_request() ??
  • [ ]Update Docs
  • [ ] Update Docstrings
  • [ ] Run pytest
  • [ ] Run tox
  • [ ] Run pylint
  • [ ] Run mypy

Review

  • [ ] Docs

Follow-Ups

  • [ ]

skasberger avatar Jan 29 '21 01:01 skasberger

As discussed during the 2024-02-14 meeting of the pyDataverse working group, we are closing old milestones in favor of a new project board at https://github.com/orgs/gdcc/projects/1 and removing issues (like this one) from those old milestones. Please feel free to join the working group! You can find us at https://py.gdcc.io and https://dataverse.zulipchat.com/#narrow/stream/377090-python

pdurbin avatar Feb 14 '24 19:02 pdurbin

I accidentally found this issue now, and it seems to be the same as #192, which has a PR at #201.

shoeffner avatar Jul 21 '24 07:07 shoeffner

Yes, thanks. I'll close this in favor of the newer issue and PR.

pdurbin avatar Jul 22 '24 14:07 pdurbin