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

feat: Add Snowflake Oauth security integration to sdk #2830

Merged
merged 7 commits into from
May 24, 2024

Conversation

sfc-gh-jmichalak
Copy link
Collaborator

@sfc-gh-jmichalak sfc-gh-jmichalak commented May 21, 2024

This change adds Snowflake Oauth security integration to sdk. Also some code is moved to helpers.

Test Plan

  • unit tests
  • integration tests

References

https://docs.snowflake.com/en/sql-reference/sql/create-security-integration
https://docs.snowflake.com/en/sql-reference/sql/alter-security-integration

Copy link

Integration tests failure for dc6431eed674ac3e1b38ab57607d1bc24cfcef89

Copy link

Integration tests failure for 8db88a7a414266122579a5d64f4283a3ca0d44c2

@sfc-gh-jmichalak sfc-gh-jmichalak marked this pull request as ready for review May 22, 2024 08:19
Copy link

Integration tests failure for a1b3413ce24f05973451d3e3ddf6fbd00db10c82

Copy link

Integration tests success for 22971e35d997ecdf0c9e072de377822777a2ff13

Copy link

Integration tests success for b37bbaf5659ce94d8db9ee5b0b733ed10c8606e2

pkg/sdk/security_integrations_def.go Outdated Show resolved Hide resolved
pkg/sdk/security_integrations_def.go Outdated Show resolved Hide resolved
pkg/sdk/security_integrations_def.go Show resolved Hide resolved
pkg/sdk/security_integrations_def.go Show resolved Hide resolved
pkg/sdk/security_integrations_def.go Outdated Show resolved Hide resolved
pkg/sdk/security_integrations_def.go Show resolved Hide resolved
pkg/sdk/security_integrations_def.go Show resolved Hide resolved
pkg/sdk/security_integrations_def.go Show resolved Hide resolved
pkg/sdk/security_integrations_dto_builders_gen.go Outdated Show resolved Hide resolved
pkg/sdk/security_integrations_def.go Show resolved Hide resolved
pkg/sdk/security_integrations_def.go Show resolved Hide resolved
pkg/sdk/security_integrations_def.go Show resolved Hide resolved
pkg/sdk/security_integrations_dto_gen.go Show resolved Hide resolved
pkg/sdk/security_integrations_gen_test.go Outdated Show resolved Hide resolved
pkg/sdk/security_integrations_gen_test.go Outdated Show resolved Hide resolved
Copy link

Integration tests success for 8b48c4048f2bfb4d9e56cda9bd3e532de1a58bde

@sfc-gh-asawicki sfc-gh-asawicki self-requested a review May 24, 2024 09:22
@sfc-gh-jmichalak sfc-gh-jmichalak merged commit b576f29 into main May 24, 2024
9 checks passed
@sfc-gh-jmichalak sfc-gh-jmichalak deleted the jmichalak-security-integrations branch May 24, 2024 09:30
sfc-gh-jcieslak pushed a commit that referenced this pull request Jun 6, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.92.0](v0.91.0...v0.92.0)
(2024-06-06)


### 🎉 **What's new:**

* Add Api Authentication security integration to sdk
([#2840](#2840))
([57a07ee](57a07ee))
* Add External Oauth security integration to sdk
([#2835](#2835))
([82d1c09](82d1c09))
* add network rules
([#2746](#2746))
([c79fa29](c79fa29))
* Add SCIM and SAML2 security integrations to sdk
([#2799](#2799))
([1312ff1](1312ff1))
* Add Snowflake Oauth security integration to sdk
([#2830](#2830))
([b576f29](b576f29))
* Database resource v1 readiness
([#2834](#2834))
([30fe136](30fe136))
* Database SDK upgrade
([#2814](#2814))
([750fe37](750fe37))


### 🔧 **Misc**

* accept non-pointer values in the generated builder methods
([#2816](#2816))
([c29fbf1](c29fbf1))
* Add a script for creating labels
([#2778](#2778))
([ce0fbad](ce0fbad))
* Adjust before 0.92.0
([#2857](#2857))
([0598656](0598656))
* Continue random ids rework
([#2819](#2819))
([f20940c](f20940c))
* Random ids rework part3
([#2833](#2833))
([36ead85](36ead85))
* Random ids rework part4
([#2837](#2837))
([64518a3](64518a3))
* Update issue for table and warehouse redesign state
([#2845](#2845))
([149e55e](149e55e))


### 🐛 **Bug fixes:**

* Fix failing integration tests
([#2832](#2832))
([2e2ca6c](2e2ca6c))
* Fix QUOTED_IDENTIFIERS_IGNORE_CASE parameter test
([#2841](#2841))
([92ad1d3](92ad1d3))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: snowflake-release-please[bot] <105954990+snowflake-release-please[bot]@users.noreply.github.com>
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.

3 participants