SUSE / container-suseconnect

Provides access to repositories inside containers using the host entitlements
Apache License 2.0
19 stars 16 forks source link

Doesn't work with our subscription / if several regcodes are returned, some of them expired #58

Closed skeller closed 1 year ago

skeller commented 2 years ago

requestRegcodes() adds all the subscriptions found to the slice. In our case the GET request returns two subscriptions, one with an expired regcode, the second with the current, active one. RequestProducts() then bails on any error returned. Of course there is a 401 returned for the expired regcode. Two ways to fix this:

Probably both should be implemented. If at least one of the regcodes worked there is no need to break the entire operation. We currently cannot use SLES containers because of this issue. I'm going to fix this locally (probably by just ignoring the error in RequestProducts()).

dirkmueller commented 1 year ago

your PR has been merged and released as 2.4.0