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

Created lambda for delete user #1206

Merged
merged 2 commits into from
Jul 26, 2022

Conversation

itbel
Copy link
Collaborator

@itbel itbel commented Jul 25, 2022

not yet functional

@itbel itbel marked this pull request as ready for review July 25, 2022 23:05

const user = await deleteCognitoUser(userID)
console.log({ user })
console.log({ env: process.env })

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information

Sensitive data returned by [process environment](1) is logged here.
@GeorgeBellTMH
Copy link
Contributor

Doesn't this functionality already exist? It should be available under edit user...

@GeorgeBellTMH
Copy link
Contributor

I had talked with Lucas...is there a ticket for this...I have more details...

@GeorgeBellTMH
Copy link
Contributor

#1207

@GeorgeBellTMH
Copy link
Contributor

Merging this because it's needed to do further updates...but we can probably delete this function

@GeorgeBellTMH GeorgeBellTMH merged commit f451d85 into jesus-collective:master Jul 26, 2022
@itbel itbel deleted the ib-delete-user branch November 30, 2022 13:54
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.

2 participants