-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fixes cisco for username and password #2267
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2267 +/- ##
==========================================
+ Coverage 79.77% 95.08% +15.3%
==========================================
Files 170 17 -153
Lines 7838 5171 -2667
Branches 1159 195 -964
==========================================
- Hits 6253 4917 -1336
+ Misses 1275 183 -1092
+ Partials 310 71 -239 |
cookie = ('APIC-Request-Signature={}; ' | ||
'APIC-Certificate-Algorithm=v1.0; ' | ||
'APIC-Certificate-Fingerprint=fingerprint; ' | ||
'APIC-Certificate-DN={}').format(signature, self.certDn) | ||
prepped_request.headers['Cookie'] = cookie | ||
else: | ||
self.log.warning("The Cisco ACI Integration requires either a cert or a username and password") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be an error instead ? Can we even continue the check if we have neither ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yea it should be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it will just error out anyway
What does this PR do?
The Cisco ACI Integration had a regression in username and password when we added certificates
Motivation
issue #2237 from @jgcasd
Review checklist
no-changelog
label attachedAdditional Notes
Anything else we should know when reviewing?