No Code Attached Yet
avatar heelc29
heelc29
4 Mar 2024

After #42945 merged, there are many "abandoned" warnings during composer install @laoneo @LadySolveig

https://ci.joomla.org/joomla/joomla-cms/74324/1/6
image

avatar heelc29 heelc29 - open - 4 Mar 2024
avatar joomla-cms-bot joomla-cms-bot - change - 4 Mar 2024
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 4 Mar 2024
avatar richard67
richard67 - comment - 4 Mar 2024

Maybe it just needs to change this line https://github.com/joomla/joomla-cms/blob/5.1-dev/composer.json#L105
from "web-token/signature-pack": "^3.2.10",
to "web-token/jwt-library": "^3.3.1",?

avatar heelc29
heelc29 - comment - 4 Mar 2024

Maybe it just needs to change this line https://github.com/joomla/joomla-cms/blob/5.1-dev/composer.json#L105 from "web-token/signature-pack": "^3.2.10", to "web-token/jwt-library": "^3.3.1"?

I just tried it 5 minutes ago ?

avatar richard67
richard67 - comment - 4 Mar 2024

Hmm, yes, seems not to help.

Maybe we can't really avoid that because it's caused by that library's composer.json files?

avatar heelc29
heelc29 - comment - 4 Mar 2024

Hmm, yes, seems not to help.

Maybe we can't really avoid that because it's caused by that library's composer.json files?

It works ... there is no longer a warning
image

avatar richard67
richard67 - comment - 4 Mar 2024

Hmm, yes, seems not to help.
Maybe we can't really avoid that because it's caused by that library's composer.json files?

It works ... there is no longer a warning

@heelc29 What did you change? I changed line 105 in the composer.json from "web-token/signature-pack": "^3.2.10", to "web-token/jwt-library": "^3.3.1", but that did not help here. If you have a fix: Could you make a PR?

avatar richard67
richard67 - comment - 4 Mar 2024

Hmm, I did now run a composer update web-token/jwt-library in addition to the mentioned change, and that seems to fix it.

avatar richard67
richard67 - comment - 4 Mar 2024

I will make a PR in a few minutes.

avatar richard67
richard67 - comment - 4 Mar 2024

@heelc29 I see you already created one ... would have been helpful to mention that here and to close this issue like we do it when we have a PR for it.

avatar richard67 richard67 - change - 4 Mar 2024
Status New Closed
Closed_Date 0000-00-00 00:00:00 2024-03-04 18:42:37
Closed_By richard67
avatar richard67 richard67 - close - 4 Mar 2024
avatar richard67
richard67 - comment - 4 Mar 2024

Closing as having a pull request. See #42954 .

avatar heelc29
heelc29 - comment - 4 Mar 2024

@heelc29 I see you already created one ... would have been helpful to mention that here and to close this issue like we do it when we have a PR for it.

Was not sure if its the "right" fix...

avatar richard67
richard67 - comment - 4 Mar 2024

I wasn't sure either, but if it is ok it will also need my follow up PR #42955 .

Add a Comment

Login with GitHub to post a comment