SUSE / rmt

RPM repository mirroring tool and registration proxy for SUSE Customer Center.
Other
37 stars 46 forks source link

Log byos only #1111

Closed jesusbv closed 3 months ago

jesusbv commented 3 months ago

Description

Prevent logging authenticated system info if instance is not BYOS

Change Type

Please select the correct option.

Checklist

Please check off each item if the requirement is met.

Other Notes

Please use this space to provide notes or thoughts to the team, such as tips on how to review/demo your changes.

jesusbv commented 3 months ago

Code LGTM, we will not hit this in the SUE operated update infrastructure since we have token sending disabled.

while we have

connect_api:
  disable_system_tokens: true

on the RMT config, BYOS instances get a system_token value which is the instance id (depending on the CSP)