Fix authentication issue for public cloud instance

SUSE / container-suseconnect

Provides access to repositories inside containers using the host entitlements

Apache License 2.0

19 stars 16 forks source link

Fix authentication issue for public cloud instance #42

Closed rhafer closed 4 years ago

rhafer commented 4 years ago

The RMT servers used by the public cloud on demand instances require authentication (using the SCCcredentials) for accessing the repositories. This commit mangles the URLs returned by RMT to add the "credentials" parameter. Which makes zypper use authentication.

Issue: #41 Signed-off-by: Ralf Haferkamp rhafer@suse.com

saschagrunert commented 4 years ago

Generally LGTM, just a single suggestion :point_up:

rhafer commented 4 years ago

LGTM, I agree with the request made by @saschagrunert to avoid the global isSMT variable.

Ok. This has been addressed now.