Skip to content
This repository has been archived by the owner on Jan 21, 2025. It is now read-only.

Authentication persistent storage GO #91

Merged
merged 23 commits into from
Nov 8, 2023

Conversation

dfinteligenz
Copy link
Contributor

Description

Add external IdP authentication for the persistent storage.

Motivation and Context

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Other changes (ci configuration, documentation or any other kind of changes)

Checklist:

  • I have created tests for my code changes, and the tests are passing.
  • I have executed the pre-commit hooks locally.
  • My change requires a change to the documentation (create a new issue if the documentation has not been updated).
  • I have updated the documentation accordingly.

@dfinteligenz dfinteligenz self-assigned this Nov 3, 2023
@dfinteligenz dfinteligenz requested review from kafkaphoenix, alexrodfe and JYisus and removed request for kafkaphoenix November 3, 2023 15:55
@dfinteligenz dfinteligenz marked this pull request as draft November 3, 2023 15:55
Copy link

github-actions bot commented Nov 3, 2023

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

github-actions bot commented Nov 6, 2023

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

github-actions bot commented Nov 6, 2023

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@dfinteligenz dfinteligenz changed the title Aauthentication persistent storage Authentication persistent storage GO Nov 6, 2023
@dfinteligenz dfinteligenz marked this pull request as ready for review November 6, 2023 13:39
Copy link

github-actions bot commented Nov 7, 2023

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

1 similar comment
Copy link

github-actions bot commented Nov 7, 2023

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions github-actions bot added size/m and removed size/xl labels Nov 7, 2023
@dfinteligenz dfinteligenz requested a review from JYisus November 7, 2023 19:14
JYisus
JYisus previously approved these changes Nov 8, 2023
@github-actions github-actions bot added size/l and removed size/m labels Nov 8, 2023
Copy link

sonarqubecloud bot commented Nov 8, 2023

[KAI - Python SDK] 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 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link

sonarqubecloud bot commented Nov 8, 2023

[AIO - GO SDK] 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 0 Code Smells

85.4% 85.4% Coverage
0.0% 0.0% Duplication

@dfinteligenz dfinteligenz requested a review from JYisus November 8, 2023 14:41
@dfinteligenz dfinteligenz enabled auto-merge (squash) November 8, 2023 14:41
@dfinteligenz dfinteligenz merged commit deae892 into develop Nov 8, 2023
9 checks passed
@dfinteligenz dfinteligenz deleted the feature/authentication-persistent-storage branch November 8, 2023 15:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants