CVE-2021-21411 (Go, published 2021-03-26T21:15Z)
Description: value: OAuth2-Proxy is an open source reverse proxy that provides authentication with Google, Github or other providers. The `--gitlab-group` flag for group-based authorization in the GitLab provider stopped working in the v7.0.0 release. Regardless of the flag settings, authorization wasn't restricted. Additionally, any authenticated users had whichever groups were set in `--gitlab-group` added to the new `X-Forwarded-Groups` header to the upstream application. While adding GitLab project based authorization support in #630, a bug was introduced where the user session's groups field was populated with the `--gitlab-group` config entries instead of pulling the individual user's group membership from the GitLab Userinfo endpoint. When the session groups where compared against the allowed groups for authorization, they matched improperly (since both lists were populated with the same data) so authorization was allowed. This impacts GitLab Provider users who relies on group membership for authorization restrictions. Any authenticated users in your GitLab environment can access your applications regardless of `--gitlab-group` membership restrictions. This is patched in v7.1.0. There is no workaround for the Group membership bug. But `--gitlab-project` can be set to use Project membership as the authorization checks instead of groups; it is not broken.
Severity: MEDIUM (CVSS v3 base score: 5.5)
Weakness: CWE-863 \u2014 Incorrect Authorization
The product performs an authorization check when an actor attempts to access a resource or perform an action, but it does not correctly perform the check. This allows attackers to bypass intended access restrictions.
Repository: https://github.com/oauth2-proxy/oauth2-proxy
Files changed:
providers/gitlab.go
providers/gitlab_test.go
Commit message: Merge pull request from GHSA-652x-m2gr-hppm

* Populate session Groups from userinfo response

* Fix: gitlab tests

Co-authored-by: Wilfried OLLIVIER <wollivier@bearstech.com>

Vulnerable code:

	p.addGroupsToSession(ctx, s)

}

// addGroupsToSession projects into session.Groups
func (p *GitLabProvider) addGroupsToSession(ctx context.Context, s *sessions.SessionState) {
	// Iterate over projects, check if oauth2-proxy can get project information on behalf of the user
	for _, group := range p.Groups {
		s.Groups = append(s.Groups, fmt.Sprintf("group:%s", group))
	}
		} else {
			logger.Errorf("Warning: project %s is archived", project.Name)




				expectedValue: []string{"project:my_group/my_project"},
				expectedValue: nil,
				expectedValue: []string{"project:my_profile/my_personal_project"},
				expectedValue: nil,
				expectedValue: nil,
				expectedValue: []string{"group:foo"},
				expectedValue: []string{"group:foo", "group:baz", "project:my_group/my_project", "project:my_profile/my_personal_project"},
				expectedValue: nil,

Fixed code:
	for _, group := range userInfo.Groups {
		s.Groups = append(s.Groups, fmt.Sprintf("group:%s", group))
	}
			continue
		logger.Errorf("Warning: project %s is archived", project.Name)
				expectedValue: []string{"group:foo", "group:bar", "project:my_group/my_project"},
				expectedValue: []string{"group:foo", "group:bar"},
				expectedValue: []string{"group:foo", "group:bar", "project:my_profile/my_personal_project"},
				expectedValue: []string{"group:foo", "group:bar"},
				expectedValue: []string{"group:foo", "group:bar"},
				expectedValue: []string{"group:foo", "group:bar"},
				expectedValue: []string{"group:foo", "group:bar", "project:my_group/my_project", "project:my_profile/my_personal_project"},
				expectedValue: []string{"group:foo", "group:bar"},