Project

General

Profile

Development #81303

ne pas signer les appels qui contiennent une authent HTTP

Added by Frédéric Péters 10 days ago. Updated 9 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
18 September 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

On aimerait bien utiliser l'authent HTTP mais pour les appels webservice qui s'adressent à wcs on va être sur un service connu, ça va ajouter la signature automatiquement, et ensuite dans get_user_from_api_query_string c'est la signature qui va être prise en compte :

def get_user_from_api_query_string(api_name=None):
    # check signature or auth header
    if not is_url_signed():
        user = getattr(get_request(), 'user', None)
        if user and user.is_api_user:
            return user
    ...

et l'appel sera signé mais il ne sera associé à personne, et on se trouvera à créer un compte utilisateur pour ajouter un ?email= et après un an le compte sera supprimé pour inactivité et l'appel ne marchera plus et on perdra du temps à retrouver les petits.

Pour avancer, petite étape, je serais pour ne plus chercher à auto-signer les appels dont l'URL contient une auth HTTP.

Associated revisions

Revision e91f503e (diff)
Added by Frédéric Péters 10 days ago

wscalls: do not auto sign URLs using HTTP basic authentication (#81303)

History

#1

Updated by Robot Gitea 10 days ago

  • Status changed from Nouveau to En cours

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#2

Updated by Robot Gitea 10 days ago

  • Status changed from En cours to Solution proposée
#3

Updated by Robot Gitea 9 days ago

  • Status changed from Solution proposée to Solution validée

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#4

Updated by Robot Gitea 9 days ago

  • Status changed from Solution validée to Résolu (à déployer)

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#5

Updated by Transition automatique 9 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF