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

vip app deploy - Allow passing in a deploy key with env variable #1718

Merged
merged 13 commits into from
Mar 6, 2024

Conversation

rebeccahum
Copy link
Contributor

@rebeccahum rebeccahum commented Mar 5, 2024

Description

This allows us to pass in the Custom Deploy Key into vip app deploy.

Also refactors API() to allow using a customAuthToken. Removes headers in HttpLink() to allow for the authLink to dynamically set by context.

Pull request checklist

New release checklist

Steps to Test

  1. Generate Custom Deploy Key
  2. Pass that in via CUSTOM_DEPLOY_KEY=$token vip @<env id>.production app deploy ~/Desktop/vip-go-skeleton.zip
  3. See prompt for confirmation on homeurl
  4. Also test to make sure that a bad key throws an unauthorized error

@rebeccahum rebeccahum force-pushed the add/passing-byor-key branch from 610ff24 to 3a20309 Compare March 5, 2024 21:47
@rebeccahum rebeccahum marked this pull request as ready for review March 5, 2024 21:56
@rebeccahum rebeccahum requested review from a team and chriszarate March 5, 2024 22:00
src/lib/api.ts Outdated Show resolved Hide resolved
@rebeccahum rebeccahum force-pushed the add/passing-byor-key branch 2 times, most recently from 5415f83 to 478ef8c Compare March 5, 2024 22:46
@rebeccahum rebeccahum force-pushed the add/passing-byor-key branch from 478ef8c to 7cd703c Compare March 5, 2024 22:48
@rebeccahum rebeccahum force-pushed the add/passing-byor-key branch 6 times, most recently from 331bc30 to 5cbeb31 Compare March 5, 2024 23:11
src/lib/api.ts Outdated Show resolved Hide resolved
@rebeccahum rebeccahum force-pushed the add/passing-byor-key branch from ddd911a to c1749e9 Compare March 6, 2024 13:53
@rebeccahum rebeccahum force-pushed the add/passing-byor-key branch from f1b50f6 to 593df96 Compare March 6, 2024 13:56
@rebeccahum rebeccahum force-pushed the add/passing-byor-key branch from 72ea73e to a642ccc Compare March 6, 2024 14:46
@rebeccahum rebeccahum force-pushed the add/passing-byor-key branch from d17f8fc to 305ee4e Compare March 6, 2024 19:17
Copy link

sonarqubecloud bot commented Mar 6, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@rebeccahum rebeccahum merged commit 23b615f into trunk Mar 6, 2024
16 checks passed
@rebeccahum rebeccahum deleted the add/passing-byor-key branch March 6, 2024 19:44
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.

4 participants