-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
docs: /api/plan
and /api/apply
endpoints
#3013
Conversation
I couldn't find out how to serve OpenAPI spec on vuepress, so I resorted to plain markdown file for the API doc. 😞 |
/api/*
endpoints/api/plan
and /api/apply
endpoints
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
I'm using : ./atlantis server --config=atlantis-server-config.yaml and on
that yaml I have `api-secret`
…On Tue, Jan 24, 2023 at 3:59 PM Li Lin ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In runatlantis.io/docs/server-side-repo-config.md
<#3013 (comment)>
:
> @@ -78,10 +78,6 @@ repos:
post_workflow_hooks:
- run: my-post-workflow-hook-command arg1
- # api-secret defines the secret used to pass to the /api/plan and /api/apply endpoints
- # if not present the api endpoints will not be accesible.
- api-secret: "myapisecret"
@jamengual <https://github.com/jamengual> Hmm... is it possible that you
have passed ATLANTIS_API_SECRET environment variable when you tested it?
If api-secret is not defined as part of the Repo
<https://github.com/runatlantis/atlantis/blob/main/server/core/config/raw/global_cfg.go#L22>
struct, Atlantis cannot parse the value, right? 🤔
—
Reply to this email directly, view it on GitHub
<#3013 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAQ3ERBV4SVVSS6WEOPOCT3WUBUGHANCNFSM6AAAAAAT72QW3E>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@jamengual The error
|
so
|
so it is not part of the Repo config that was my mistake, it is a server flag. |
what
why
The feature has become stable and was lacking an API doc.
references