[BUG][0.3.9] Authentication method concern due to deprecation of token in API’s query string for Memsource TMS APIs

[cajieh]

Describe the bug
Find out the impact of the deprecation and how to change the authentication method to one of the supported ones.

Affected version

To Reproduce
Steps to reproduce the behavior

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots or Traceback
If applicable, add screenshots or Traceback to help explain your problem.

Additional context
Add any other context about the problem here.

[zerodayz]

Hi @cyril-ui-developer

It needs to be changed https://github.com/unofficial-memsource/memsource-cli-client/blob/master/memsource_cli/api_client.py#L103

Add something like:

if query_params['token']:
    header_params.append('Authorization', "Bearer " + query_params['token'])
    query_params.remove('token') 

Would you like to take this? or maybe @ludekjanda can change it.

[cajieh]

Hi @cyril-ui-developer

It needs to be changed https://github.com/unofficial-memsource/memsource-cli-client/blob/master/memsource_cli/api_client.py#L103

Add something like:

if query_params['token']:
    header_params.append('Authorization', "Bearer " + query_params['token'])
    query_params.remove('token') 

Would you like to take this? or maybe @ludekjanda can change it.

@zerodayz Feel free to take it or assign it to @ludekjanda. I am not familiar with the codebase. Thanks.