Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Sprint 92] ISY-653 HttpInvoker setzt Authorization Header nicht wenn isy security verwendet wird #187

Merged
merged 7 commits into from
Feb 5, 2024
Merged

[Sprint 92] ISY-653 HttpInvoker setzt Authorization Header nicht wenn isy security verwendet wird #187

merged 7 commits into from
Feb 5, 2024

Conversation

sb-sfink
Copy link
Contributor

@sb-sfink sb-sfink commented Dec 8, 2023

feature/ISY-653-HttpInvoker-setzt-Authorization-Header-nicht-wenn-isy-security-verwendet-wird

  • feat: ISY-653 extend TimeoutWiederholungHttpInvoker to handle missing AufrufKontextVerwalter and retrieve token from Security Context
  • chore: ISY-653 update changelogs
  • test: ISY-653 add test in TimeoutWiederholungHttpInvokerRequestExecutorTest for SecurityContext

@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Dec 8, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

88.9% 88.9% Coverage
0.0% 0.0% Duplication

@BerghoffP BerghoffP added the medium Issues/PRs with medium priority must be processed before operations with low label label Dec 11, 2023
@huebnercl
Merge branch 'master' into feature/ISY-653-HttpInvoker-setzt-Authoriz…
7a55fab 
…ation-Header-nicht-wenn-isy-security-verwendet-wird
huebnercl
huebnercl previously approved these changes Feb 5, 2024
View reviewed changes
@huebnercl
Merge branch 'master' into feature/ISY-653-HttpInvoker-setzt-Authoriz…
76b3243 
…ation-Header-nicht-wenn-isy-security-verwendet-wird
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Feb 5, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

1 New issue
0 Security Hotspots
91.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@dkurowski-materna dkurowski-materna merged commit a352969 into master Feb 5, 2024
8 checks passed
@dkurowski-materna dkurowski-materna deleted the feature/ISY-653-HttpInvoker-setzt-Authorization-Header-nicht-wenn-isy-security-verwendet-wird branch February 5, 2024 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jsawatzk jsawatzk jsawatzk left review comments

@dkurowski-materna dkurowski-materna dkurowski-materna approved these changes

@huebnercl huebnercl huebnercl left review comments

Assignees
No one assigned
Labels
medium Issues/PRs with medium priority must be processed before operations with low label
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@sb-sfink @huebnercl @dkurowski-materna @jsawatzk @BerghoffP @roman2906