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

Adding new flutter mobile reference app repo #50

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

Conversation

chaitanya-s-affinidi
Copy link

Added new repo which contains

  • Frontend - Flutter mobile app
  • Backend - Nodejs PKCE code generator and verifier
  • Deeplink - Sample repo to host the assetlinks.json file inorder to create applink in android device

Comment on lines +39 to +56
app.get('/api/affinidi-auth/init', async (req, res, next) => {

const state = generators.state();
//const code_verifier = generators.codeVerifier();
const params = {
code_challenge: generators.codeChallenge(code_verifier),
code_challenge_method: 'S256',
response_type: 'code',
scope: 'openid',
state,
}

const client = await getClient();

const authorizationUrl = client.authorizationUrl(params);

res.send({ authorizationUrl });
});

Check failure

Code scanning / CodeQL

Missing rate limiting High

This route handler performs
authorization
, but is not rate-limited.
Comment on lines +39 to +56
app.get('/api/affinidi-auth/init', async (req, res, next) => {

const state = generators.state();
//const code_verifier = generators.codeVerifier();
const params = {
code_challenge: generators.codeChallenge(code_verifier),
code_challenge_method: 'S256',
response_type: 'code',
scope: 'openid',
state,
}

const client = await getClient();

const authorizationUrl = client.authorizationUrl(params);

res.send({ authorizationUrl });
});

Check failure

Code scanning / CodeQL

Missing rate limiting High

This route handler performs
authorization
, but is not rate-limited.
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.

1 participant