fix(graphql-js): allow graphql requests with no access token (#2555)

This commit is contained in:
Hassan Ben Jobrane
2024-02-23 21:09:45 +01:00
committed by GitHub
parent 35010353c7
commit 2d68fee54c
2 changed files with 14 additions and 9 deletions

View File

@@ -0,0 +1,5 @@
---
'@nhost/graphql-js': patch
---
fix: resolve an issue where unauthenticated graphql requests are not sent

View File

@@ -30,9 +30,9 @@ export class NhostGraphqlClient {
this.adminSecret = adminSecret
}
private isAccessTokenValid = () => {
private isAccessTokenValidOrNull = () => {
if (!this.accessToken) {
return false
return true
}
try {
@@ -44,21 +44,21 @@ export class NhostGraphqlClient {
}
}
private awaitForValidAccessToken = async () => {
if (this.isAccessTokenValid()) {
private awaitForValidAccessTokenOrNull = async () => {
if (this.isAccessTokenValidOrNull()) {
return true
}
const waitForValidToken = () => {
if (this.isAccessTokenValid()) {
const waitForValidTokenOrNull = () => {
if (this.isAccessTokenValidOrNull()) {
return Promise.resolve(true)
}
return new Promise((resolve) => {
setTimeout(() => waitForValidToken().then(resolve), 100)
setTimeout(() => waitForValidTokenOrNull().then(resolve), 100)
})
}
return waitForValidToken()
return waitForValidTokenOrNull()
}
/**
@@ -106,7 +106,7 @@ export class NhostGraphqlClient {
if (!process.env.TEST_MODE) {
// We skip this while running unit tests because the accessToken is generated using faker
await this.awaitForValidAccessToken()
await this.awaitForValidAccessTokenOrNull()
}
try {