Skip to content

Addition of Gitlab authentication.#4624

Merged
bruntib merged 6 commits intoEricsson:masterfrom
feyruzb:provider-gitlab-addition
Nov 25, 2025
Merged

Addition of Gitlab authentication.#4624
bruntib merged 6 commits intoEricsson:masterfrom
feyruzb:provider-gitlab-addition

Conversation

@feyruzb
Copy link
Copy Markdown
Collaborator

@feyruzb feyruzb commented Jul 10, 2025

Added the following:

  • Gitlab OAuth template.
  • Gitlab configuration file section.
  • Documentation regarding username selection options of Gitlab.

@feyruzb feyruzb self-assigned this Jul 10, 2025
@feyruzb feyruzb added enhancement 🌟 documentation 📖 Changes to documentation. python Pull requests that update Python code (used by DependaBot) labels Jul 10, 2025
@feyruzb feyruzb force-pushed the provider-gitlab-addition branch from 2ab1fb7 to 6a19711 Compare July 10, 2025 11:48
@LorandMarton
Copy link
Copy Markdown

LorandMarton commented Jul 14, 2025 •
edited
Loading

Hi! Thanks for adding support for GitLab authentication - much appreciated!

Quick question: Will this also work with self-hosted GitLab instances? If not, would it be possible to make the template configurable (e.g., allowing a custom domain) to support self-hosted GitLab as well?

@feyruzb
Copy link
Copy Markdown
Collaborator Author

feyruzb commented Jul 14, 2025

Hi! Thanks for adding support for GitLab authentication - much appreciated!

Quick question: Will this also work with self-hosted GitLab instances? If not, would it be possible to make the template configurable (e.g., allowing a custom domain) to support self-hosted GitLab as well?

Hello, current configuration supports variables overriding, so if you specify variables mentioned below according to your custom GitLab instance in server_config file, it should work.

  • client_id
  • client_secret
  • authorization_url
  • token_url
  • user_info_url

It should look, like this:

image

@feyruzb feyruzb linked an issue Jul 17, 2025 that may be closed by this pull request
OAuth addition of Gitlab authentication #4626
Closed
@lom279
Copy link
Copy Markdown

lom279 commented Nov 12, 2025 •
edited
Loading

@bruntib @dkrupp @vodorok , this GitLab auth addition looks very useful - could it be considered for the next release?

bruntib
bruntib requested changes Nov 15, 2025
View reviewed changes
Comment thread web/server/codechecker_server/api/authentication.py Outdated
else:
username = user_info.get("mail")
elif template == "gitlab/v1":
print(user_info)
Copy link
Copy Markdown
Contributor

@bruntib bruntib Nov 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, remove this print.

Copy link
Copy Markdown
Collaborator Author

@feyruzb feyruzb Nov 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

Comment thread web/server/codechecker_server/api/authentication.py Outdated
if username_key == "username":
username = user_info.get("preferred_username")
elif username_key == "email":
print(user_info)
Copy link
Copy Markdown
Contributor

@bruntib bruntib Nov 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, remove this print.

Copy link
Copy Markdown
Collaborator Author

@feyruzb feyruzb Nov 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@feyruzb feyruzb requested a review from bruntib November 19, 2025 14:37
@bruntib bruntib merged commit 65d77e2 into Ericsson:master Nov 25, 2025
8 of 9 checks passed
@feyruzb feyruzb deleted the provider-gitlab-addition branch February 26, 2026 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bruntib bruntib bruntib approved these changes

@dkrupp dkrupp Awaiting requested review from dkrupp dkrupp is a code owner

@vodorok vodorok Awaiting requested review from vodorok vodorok is a code owner

Assignees

@feyruzb feyruzb

Labels

documentation 📖 Changes to documentation. enhancement 🌟 python Pull requests that update Python code (used by DependaBot)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

OAuth addition of Gitlab authentication

4 participants

@feyruzb @LorandMarton @lom279 @bruntib