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

check if user added to team by email #2

Merged
merged 6 commits into from
Jul 12, 2022

Conversation

jackHay22
Copy link

Changes

  • Checks if the invite email corresponds to a user that is already a member of the team

@jackHay22 jackHay22 requested a review from KN4CK3R July 12, 2022 14:17
models/organization/team_invite.go Outdated Show resolved Hide resolved
models/organization/team_invite.go Outdated Show resolved Hide resolved
@KN4CK3R
Copy link
Owner

KN4CK3R commented Jul 12, 2022

5472372 the old formatting was easier to read.

@jackHay22 jackHay22 requested a review from KN4CK3R July 12, 2022 14:35
@KN4CK3R
Copy link
Owner

KN4CK3R commented Jul 12, 2022

Good addition, thank you.

@KN4CK3R KN4CK3R merged commit 7cce63c into KN4CK3R:feature-invite Jul 12, 2022
GiteaBot pushed a commit that referenced this pull request Apr 7, 2024
Result of `go get -u golang.org/x/net; make tidy`.

This is related to the following vulncheck warning:
```
There are 2 vulnerabilities in modules that you require that are
neither imported nor called. You may not need to take any action.
See https://pkg.go.dev/golang.org/x/vuln/cmd/govulncheck for details.

Vulnerability #1: GO-2024-2687
    HTTP/2 CONTINUATION flood in net/http
  More info: https://pkg.go.dev/vuln/GO-2024-2687
  Module: golang.org/x/net
    Found in: golang.org/x/[email protected]
    Fixed in: golang.org/x/[email protected]

Vulnerability #2: GO-2022-0470
    No access control in github.com/blevesearch/bleve and bleve/v2
  More info: https://pkg.go.dev/vuln/GO-2022-0470
  Module: github.com/blevesearch/bleve/v2
    Found in: github.com/blevesearch/bleve/[email protected]
    Fixed in: N/A
```
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