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.

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

ARC-1217: GesundheitsID Simple Test Flow #7

Merged
merged 4 commits into from
Feb 1, 2024

Conversation

thomasrichner-oviva
Copy link
Contributor

No description provided.

Copy link
Contributor

@michelealbanese-oviva michelealbanese-oviva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great job 👏

README.md Outdated
# Keycloak Identity Provider for GesundheitsID (eHealthID)

## Contents
- [gesundheitsid](./gesundheitsid) - A plain Java library to build RelyingParties for GesundheitsID with.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- [gesundheitsid](./gesundheitsid) - A plain Java library to build RelyingParties for GesundheitsID with.
- [gesundheitsid](./gesundheitsid) - A plain Java library to build RelyingParties for GesundheitsID.

README.md Outdated

### Gematik Test Sektoraler IdP in Browser

Since the Gematik Referenze IDP in the Test Environment needs a custom header, it can not be used directly in the browser for authentication.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Since the Gematik Referenze IDP in the Test Environment needs a custom header, it can not be used directly in the browser for authentication.
Since the Gematik Reference IDP in the Test Environment needs a custom header, it can not be used directly in the browser for authentication.

"entity statement of '%s' has no authorization url configuration".formatted(sub));
}

public static RuntimeException missingPARUrl(String sub) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
public static RuntimeException missingPARUrl(String sub) {
public static RuntimeException missingParUrl(String sub) {

inconsistent casing (see invalidParRequestUri)

@JsonProperty("urn:telematik:claims:profession") String telematikProfession,
@JsonProperty("urn:telematik:claims:given_name") String telematikGivenName,

/** vor insured person (IP) the immutable part of the Krankenversichertennummer (KVNR) * */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
/** vor insured person (IP) the immutable part of the Krankenversichertennummer (KVNR) * */
/** for insured person (IP) the immutable part of the Krankenversichertennummer (KVNR) * */

}

public Builder requirePushedAuthorizationRequests(
Boolean requirePushedAuthorizationRequests) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Boolean requirePushedAuthorizationRequests) {
boolean requirePushedAuthorizationRequests) {

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, that's an optional field :)

Copy link

sonarqubecloud bot commented Feb 1, 2024

Quality Gate Passed Quality Gate passed

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

2 New issues
0 Security Hotspots
81.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@thomasrichner-oviva thomasrichner-oviva merged commit bcba63a into main Feb 1, 2024
7 checks passed
@thomasrichner-oviva thomasrichner-oviva deleted the feature/arc-1217 branch February 8, 2024 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants