Project

General

Profile

Development #24022

Suppression de la limite de 128 caractères pour les URLs d'un service CAS

Added by Benjamin Renard almost 2 years ago. Updated 7 days ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Gestion d'identité
Category:
authentic2-idp-cas
Target version:
-
Start date:
23 May 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Cette limite est problématique lorsque l'on a plusieurs URLs, ne serait-se que 3 ou 4. A mon sens, elle n'est pas nécessaire sachant que le champs en base est de type texte désormais. Ci-joint, un patch pour supprimer cette limitation dans le modèle. Le patch suffit pour qu'Authentic permette de dépasser cette limite. Je ne pense pas qu'une migration soit nécessaire mais à défaut, il faudra la prévoir.

0001-Remove-CAS-service-s-URLs-length-limit.patch View (1.01 KB) Benjamin Renard, 23 May 2018 12:24 PM

0001-Remove-CAS-service-s-URLs-length-limit.patch View (4.14 KB) Paul Marillonnet, 31 Mar 2020 10:53 AM

Associated revisions

Revision 46b58e26 (diff)
Added by Paul Marillonnet 8 days ago

idp_cas: drop length limit to service urls (#24022)

History

#1 Updated by Benjamin Dauvergne almost 2 years ago

Avec un test, et c'est ok (file moi ta clé publique que je t'ouvre un droit de commit, la seule règle sera d'ajouter un header License: MIT en bas de tes messages de commit).

#2 Updated by Thomas Noël almost 2 years ago

Je pense qu'on devrait/devra modifier la migration 0001 pour y supprimer le max_length

#3 Updated by Paul Marillonnet 8 days ago

Allez zou, avec un test.

#4 Updated by Benjamin Dauvergne 8 days ago

  • Status changed from Solution proposée to Solution validée
  • Tracker changed from Bug to Development

#5 Updated by Paul Marillonnet 8 days ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 46b58e26ba51893f5235ff8e8c5c3335dbb95b59
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Tue Mar 31 11:17:13 2020 +0200

    idp_cas: drop length limit to service urls (#24022)

#6 Updated by Frédéric Péters 7 days ago

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

Also available in: Atom PDF