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 secret for java sdk #42

Merged
merged 13 commits into from
Sep 29, 2022
Merged

feat: add secret for java sdk #42

merged 13 commits into from
Sep 29, 2022

Conversation

fft0518
Copy link
Contributor

@fft0518 fft0518 commented Sep 25, 2022

The PR is add secret api for java sdk and is one of the ospp projects.

In this PR, I have a problem that I don't how to convert SecretResponse type to Map type. I also use the SecretResponse type although I think it's wrong.

mosn/layotto#626

@ZLBer
Copy link
Contributor

ZLBer commented Sep 26, 2022

image

@fft0518 it has method to get sercretsMap

@ZLBer
Copy link
Contributor

ZLBer commented Sep 27, 2022

@fft0518 add a java example? like examples-sequencer

@ZLBer
Copy link
Contributor

ZLBer commented Sep 27, 2022

@fft0518 merge the master branch before push code
image

@fft0518
Copy link
Contributor Author

fft0518 commented Sep 27, 2022

@fft0518 add a java example? like examples-sequencer
I added it at the first time.

ZLBer
ZLBer previously approved these changes Sep 27, 2022
@fft0518
Copy link
Contributor Author

fft0518 commented Sep 27, 2022

@seeflood please help to review it. Thanks.

examples-secret/pom.xml Outdated Show resolved Hide resolved
examples-secret/pom.xml Outdated Show resolved Hide resolved
examples-secret/pom.xml Show resolved Hide resolved
Copy link
Member

@seeflood seeflood left a comment

Choose a reason for hiding this comment

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

lgtm. Thanks

@seeflood seeflood merged commit 165f6f3 into layotto:main Sep 29, 2022
@fft0518
Copy link
Contributor Author

fft0518 commented Sep 29, 2022

lgtm. Thanks

Thanks!

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.

[OSPP 2022] improve layotto-java-sdk and layotto-spring-boot
3 participants