Caveats in README

Missing actual maven command

mvn -Dokta.oauth2.issuer=https://{yourOktaDomain}/oauth2/default \
    -Dokta.oauth2.clientId={clientId} \
    -Dokta.oauth2.clientSecret={clientSecret} \
    -Dokta.oauth2.postLogoutRedirectUri={absoluteLogoutRedirectUri} 

• Turn out it’s missing spring-boot:run to the end.
• one nit: omit optional parameters and may have another section to describe what are all supported parameters

How to update application.yml

NOTE: Putting secrets on the command line should ONLY be done for examples, do NOT do this in production. Instead update the projects application.yml

How to update application.yml? Are we assuming developers has enough knowledge of sprint-boot? I wonder if it makes sense to just add example (for copy/parse)? It doesn’t hurt even developer know how to do not to mention some one dont know (like me) and I have to google…

Thanks!

[bdemers]

@haishengwu-okta no, I think it’s a valid point. I’m not a big fan of the multiple examples in one repo, it makes it harder to grab just that one example.

I do like the consistency between projects, but I think we could try to call this out more.

[haishengwu-okta]

Thanks @bdemers for helping my mistake! I’ll close the ticket and leave the application.yml whether you like to add sample config.