search

Unicon / shibboleth-idp-dockerized

A Shibboleth Identity Provider (IdP) base-image
Apache License 2.0
108 stars 54 forks source link

CLI broken: aacli and status #65

Open vwbusguy opened 5 years ago

vwbusguy commented 5 years ago

In the 3.4.3 image, attempting to run bin/aacli.sh, bin/status.sh, or version.sh returns an error: Error: Could not find or load main class net.shibboleth.idp.cli.CLI, even with just invoking help. I'm running these with Shibboleth running - I can fetch the metadata page remotely and the /status endpoint is live and appears normal.

We use aacli.sh as part of our integration testing. I additionally, verified that ResolverTest is enabled in the conf/admin/general-admin.xml file. The same config works in the 3.3.2 image.

jtgasper3 commented 5 years ago

@vwbusguy ,

You are correct. The 3.4.x shell script's classpath was updated to point libraries in a new location that the installer copies libs into. I haven't had a chance to move the libs into the same spot via the Dockerfile.

Definitely a todo. For now, you can mod the runclass.sh (guessing at the name) to remove dist (also guessing at the value) and saving the file. The utilities scripts should then work.

John

vwbusguy commented 5 years ago

The missing dist might be related to https://github.com/Unicon/shib-cas-authn3/issues/39 as well.

vwbusguy commented 5 years ago

Removing dist still complains about the missing class. I assume I'll also need to update the classpath as well.

jtgasper3 commented 5 years ago

Removing the directory doesn’t help, the lib directory needs to get moved into it (or similar) OR change the classpath to point to the old location.

From: Scott Williams notifications@github.com Sent: Tuesday, June 18, 2019 1:44 PM To: Unicon/shibboleth-idp-dockerized shibboleth-idp-dockerized@noreply.github.com Cc: John Gasper jgasper@unicon.net; Comment comment@noreply.github.com Subject: Re: [Unicon/shibboleth-idp-dockerized] CLI broken: aacli and status (#65)

Removing dist still complains about the missing class. I assume I'll also need to update the classpath as well.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Unicon/shibboleth-idp-dockerized/issues/65?email_source=notifications&email_token=AAHZULHZEN7CJYPQDWSREIDP3FCJ7A5CNFSM4HZC5XKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX75AZA#issuecomment-503304292 , or mute the thread https://github.com/notifications/unsubscribe-auth/AAHZULGELLGMM4JMVLDM3ZTP3FCJ7ANCNFSM4HZC5XKA . https://github.com/notifications/beacon/AAHZULC6Q5HJNCWKLPKVROLP3FCJ7A5CNFSM4HZC5XKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX75AZA.gif

This email has been scanned for spam and viruses by Proofpoint Essentials. Click here https://us2.proofpointessentials.com/index01.php?mod_id=11&mod_option=logitem&mail_id=1560890656-wf6-pSVBkwSZ&r_address=jgasper%40unicon.net&report=1 to report this email as spam.

vwbusguy commented 5 years ago

It looks like setting this in runclass.sh allowed it to work: LOCALCLASSPATH="$LOCATION/../webapp/WEB-INF/lib/*":$LOCALCLASSPATH

vwbusguy commented 5 years ago

I was able to override it for now by overwriting the runclass.sh during docker build. Our automated tests against aacli.sh are working again. Many thanks. For now, we'll compare for changes with each tag change. Please let me know if you would like any help testing a more permanent fix.