Skip to content
This repository has been archived by the owner on Feb 21, 2022. It is now read-only.

feat: add useApiTokenAuth config #362

Open
wants to merge 7 commits into
base: hangar
Choose a base branch
from
Open

Conversation

imunitic
Copy link
Contributor

This enables us to switch on/off authentication for the
graphql backend as on production we use Googles IAP

Ivica Munitic added 3 commits February 12, 2021 11:31
This enables us to switch on/off authentication for the
graphql backend as on production we use Googles IAP
@lgtm-com
Copy link

lgtm-com bot commented Feb 12, 2021

This pull request introduces 1 alert when merging fb6a91f into d8f041d - view on LGTM.com

new alerts:

  • 1 for Syntax error

@sonarcloud
Copy link

sonarcloud bot commented Feb 12, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

frontend/packages/zoo-api/src/client.ts Outdated Show resolved Hide resolved
@@ -1,6 +1,7 @@
app:
title: Hangar App
baseUrl: http://localhost:3000
useApiTokenAuth: false
Copy link
Contributor

Choose a reason for hiding this comment

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

Does it work for you? It seems like we’d need to change the schema of the config somewhere.
Anyway, it doesn’t pick the config when I’m trying locally 😕

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I still have to successfully run this locally.

@ghost
Copy link

ghost commented Feb 15, 2021

Congratulations 🎉. DeepCode analyzed your code in 4.949 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants