OpenConext / Stepup-Middleware

Stepup Middleware
Apache License 2.0
3 stars 2 forks source link

Dependency illegitimately listed under require-dev #411

Open tvdijen opened 11 months ago

tvdijen commented 11 months ago

The following dependency are listed under require-dev in composer.json but turns out to be a hard dependency required to use the stepup-suite:

To be able to register and activate a token in selfservice, the following dependency is mandatory:

STEPUP-MIDDLEWARE[163023]: {"channel":"app","level":"CRITICAL","message":"Attempted to load class \"Stopwatch\" from namespace \"Symfony\\Component\\Stopwatch\".\nDid you forget a \"use\" statement for another namespace?","context":{"exception":{"class":"Symfony\\Component\\ErrorHandler\\Error\\ClassNotFoundError","message":"Attempted to load class \"Stopwatch\" from namespace \"Symfony\\Component\\Stopwatch\".\nDid you forget a \"use\" statement for another namespace?","code":0,"file":"/var/cache/stepup/middleware/ContainerQ6QC8oP/srcApp_KernelProdContainer.php:2043"}},"extra":{"server":"middleware.stepup-prd.xxx.nl","application":"middleware","request_id":"dca7608edb4be1fa13f4a5452306ee97"}}

Now, when I move symfony/stopwatch to the required dependencies, I can finish the registration proces... Installing Middleware without the --no-dev also solves this issues, but is of course undesireable for many reasons

phavekes commented 11 months ago

symfony/web-profiler-bundle should never be used in production

@MKodde Can you discover why this error occurs?

tvdijen commented 11 months ago

I see this when installing v5.0.10 with the --no-dev flag.

tvdijen commented 11 months ago

I've updated the issue description because some issues were caused by a missing environment variable. Another one was a session related issue, but the one issue above however is still valid.