search

DIRACGrid / diracx

The neXt DIRAC incarnation
GNU General Public License v3.0
9 stars 20 forks source link

Switch to EdDSA for JWTs in tests #279

Closed fstagni closed 2 months ago

chaen commented 2 months ago

This tremendously slows down the tests, that's why it's so low. What library requires 1024 ?

martynia commented 2 months ago

See issue #278. J

On 16 Aug 2024, at 15:25, chaen @.***> wrote:

This tremendously slows down the tests, that's why it's so low. What library requires 1024 ?

— Reply to this email directly, view it on GitHub https://github.com/DIRACGrid/diracx/pull/279#issuecomment-2293506874, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB2KE7J5TSMQ7PLNGG76F5TZRX4T5AVCNFSM6AAAAABMT4HA3SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJTGUYDMOBXGQ. You are receiving this because you are subscribed to this thread.

martynia commented 2 months ago

Can we merge this, or are we waiting for an alternative solution (if it exists)?

fstagni commented 2 months ago

I'd merge this for now and look for better solution later on. @chaen @chrisburr ?

chaen commented 2 months ago

you have the same fixes in https://github.com/DIRACGrid/diracx/pull/281 that we commented on

fstagni commented 2 months ago

you have the same fixes in #281 that we commented on

You mean this? https://github.com/DIRACGrid/diracx/pull/281#discussion_r1728853076

I am waiting for a reply.

chrisburr commented 2 months ago

And https://github.com/DIRACGrid/diracx/pull/281#discussion_r1728853721 And https://github.com/DIRACGrid/diracx/pull/281#discussion_r1729005018