Skip to content

Allow access to Users Public GPG Keys

Michelle Gill requested to merge api-expose-gpg into master

What does this MR do?

This change will make the public keys of an individual user accessible without public authentication.

From a security perspective !19952 (comment 386792881):

@m_gill This is OK from a security point of view as public keys are by definition public and according to the docs it does not return any another metadata about the user

Started from a community contribution that would require read_only permissions to access this endpoint, but since these are public keys and the issue itself seems more geared towards pasting links for anyone to access, I changed this to public authentication.

Screenshots

Before: Without token, or without permission to user After: Without token, or without permission to user
Screen_Shot_2020-09-29_at_4.46.09_AM Screen_Shot_2020-09-29_at_4.51.05_AM

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Michelle Gill

Merge request reports