fix(gitlab): Fix auth error when trying to sync public gitlab.com project with no token

[msukkari]

• undefined check for token when initializing gitlab api object
• fixes incorrect logic for isGitlabCloud and isGitHubCloud
• fixes minor bugs for log statements

Summary by CodeRabbit

Bug Fixes

• Fixed authentication errors when attempting to sync public projects without a configured authentication token
• Improved default authentication handling for public cloud services when custom configurations are not provided

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[msukkari]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

This PR modifies authentication defaults for GitHub and GitLab client factories, changing cloud service detection to default to true when no custom URL is provided, and conditionally includes authentication tokens only when present. Includes a CHANGELOG entry documenting an authentication fix and logger method corrections.

Changes

Cohort / File(s)	Summary
Documentation CHANGELOG.md	Added entry under Unreleased/Fixed describing authentication error fix for syncing public gitlab.com projects with no token
GitHub Client Factory packages/backend/src/github.ts	Changed isGitHubCloud default from false to true when no URL is provided in createOctokitFromToken
GitLab Client Factories packages/backend/src/gitlab.ts	Changed isGitLabCloud default from false to true when no URL is provided in both createGitLabFromPersonalAccessToken and createGitLabFromOAuthToken; conditionally include token/oauthToken in Gitlab constructor only if present; updated logger method calls from logger.warning to logger.warn
Repository Permission Syncer packages/backend/src/ee/repoPermissionSyncer.ts	Changed GitHub cloud detection to default isGitHubCloud to true when credentials.hostUrl is absent

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• fix(worker): Fix user driven permission syncing for self-hosted code hosts #729: Modifies cloud detection logic in code-host client factories (createOctokitFromToken, createGitLabFromOAuthToken) with URL-based determination patterns similar to this PR's changes.
• feat(ee): GitLab permission syncing #585: Updates GitLab and GitHub client-creation logic, directly overlapping with the client factory modifications in this PR.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main fix: addressing authentication errors when syncing public GitLab projects without a token, which aligns with the code changes (undefined token checks and isGitLabCloud defaults).
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[msukkari]

• undefined check for token when initializing gitlab api object
• fixes incorrect logic for isGitlabCloud and isGitHubCloud
• fixes minor bugs for log statements

Summary by CodeRabbit

Bug Fixes

• Fixed authentication errors when attempting to sync public projects without a configured authentication token
• Improved default authentication handling for public cloud services when custom configurations are not provided

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[msukkari]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

This PR modifies authentication defaults for GitHub and GitLab client factories, changing cloud service detection to default to true when no custom URL is provided, and conditionally includes authentication tokens only when present. Includes a CHANGELOG entry documenting an authentication fix and logger method corrections.

Changes

Cohort / File(s)	Summary
Documentation CHANGELOG.md	Added entry under Unreleased/Fixed describing authentication error fix for syncing public gitlab.com projects with no token
GitHub Client Factory packages/backend/src/github.ts	Changed isGitHubCloud default from false to true when no URL is provided in createOctokitFromToken
GitLab Client Factories packages/backend/src/gitlab.ts	Changed isGitLabCloud default from false to true when no URL is provided in both createGitLabFromPersonalAccessToken and createGitLabFromOAuthToken; conditionally include token/oauthToken in Gitlab constructor only if present; updated logger method calls from logger.warning to logger.warn
Repository Permission Syncer packages/backend/src/ee/repoPermissionSyncer.ts	Changed GitHub cloud detection to default isGitHubCloud to true when credentials.hostUrl is absent

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• fix(worker): Fix user driven permission syncing for self-hosted code hosts #729: Modifies cloud detection logic in code-host client factories (createOctokitFromToken, createGitLabFromOAuthToken) with URL-based determination patterns similar to this PR's changes.
• feat(ee): GitLab permission syncing #585: Updates GitLab and GitHub client-creation logic, directly overlapping with the client factory modifications in this PR.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main fix: addressing authentication errors when syncing public GitLab projects without a token, which aligns with the code changes (undefined token checks and isGitLabCloud defaults).
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[msukkari]

• undefined check for token when initializing gitlab api object
• fixes incorrect logic for isGitlabCloud and isGitHubCloud
• fixes minor bugs for log statements

Summary by CodeRabbit

Bug Fixes

• Fixed authentication errors when attempting to sync public projects without a configured authentication token
• Improved default authentication handling for public cloud services when custom configurations are not provided

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[msukkari]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

This PR modifies authentication defaults for GitHub and GitLab client factories, changing cloud service detection to default to true when no custom URL is provided, and conditionally includes authentication tokens only when present. Includes a CHANGELOG entry documenting an authentication fix and logger method corrections.

Changes

Cohort / File(s)	Summary
Documentation CHANGELOG.md	Added entry under Unreleased/Fixed describing authentication error fix for syncing public gitlab.com projects with no token
GitHub Client Factory packages/backend/src/github.ts	Changed isGitHubCloud default from false to true when no URL is provided in createOctokitFromToken
GitLab Client Factories packages/backend/src/gitlab.ts	Changed isGitLabCloud default from false to true when no URL is provided in both createGitLabFromPersonalAccessToken and createGitLabFromOAuthToken; conditionally include token/oauthToken in Gitlab constructor only if present; updated logger method calls from logger.warning to logger.warn
Repository Permission Syncer packages/backend/src/ee/repoPermissionSyncer.ts	Changed GitHub cloud detection to default isGitHubCloud to true when credentials.hostUrl is absent

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• fix(worker): Fix user driven permission syncing for self-hosted code hosts #729: Modifies cloud detection logic in code-host client factories (createOctokitFromToken, createGitLabFromOAuthToken) with URL-based determination patterns similar to this PR's changes.
• feat(ee): GitLab permission syncing #585: Updates GitLab and GitHub client-creation logic, directly overlapping with the client factory modifications in this PR.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main fix: addressing authentication errors when syncing public GitLab projects without a token, which aligns with the code changes (undefined token checks and isGitLabCloud defaults).
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[msukkari]

• undefined check for token when initializing gitlab api object
• fixes incorrect logic for isGitlabCloud and isGitHubCloud
• fixes minor bugs for log statements

Summary by CodeRabbit

Bug Fixes

• Fixed authentication errors when attempting to sync public projects without a configured authentication token
• Improved default authentication handling for public cloud services when custom configurations are not provided

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[msukkari]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

This PR modifies authentication defaults for GitHub and GitLab client factories, changing cloud service detection to default to true when no custom URL is provided, and conditionally includes authentication tokens only when present. Includes a CHANGELOG entry documenting an authentication fix and logger method corrections.

Changes

Cohort / File(s)	Summary
Documentation CHANGELOG.md	Added entry under Unreleased/Fixed describing authentication error fix for syncing public gitlab.com projects with no token
GitHub Client Factory packages/backend/src/github.ts	Changed isGitHubCloud default from false to true when no URL is provided in createOctokitFromToken
GitLab Client Factories packages/backend/src/gitlab.ts	Changed isGitLabCloud default from false to true when no URL is provided in both createGitLabFromPersonalAccessToken and createGitLabFromOAuthToken; conditionally include token/oauthToken in Gitlab constructor only if present; updated logger method calls from logger.warning to logger.warn
Repository Permission Syncer packages/backend/src/ee/repoPermissionSyncer.ts	Changed GitHub cloud detection to default isGitHubCloud to true when credentials.hostUrl is absent

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• fix(worker): Fix user driven permission syncing for self-hosted code hosts #729: Modifies cloud detection logic in code-host client factories (createOctokitFromToken, createGitLabFromOAuthToken) with URL-based determination patterns similar to this PR's changes.
• feat(ee): GitLab permission syncing #585: Updates GitLab and GitHub client-creation logic, directly overlapping with the client factory modifications in this PR.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main fix: addressing authentication errors when syncing public GitLab projects without a token, which aligns with the code changes (undefined token checks and isGitLabCloud defaults).
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[msukkari]

• undefined check for token when initializing gitlab api object
• fixes incorrect logic for isGitlabCloud and isGitHubCloud
• fixes minor bugs for log statements

Summary by CodeRabbit

Bug Fixes

• Fixed authentication errors when attempting to sync public projects without a configured authentication token
• Improved default authentication handling for public cloud services when custom configurations are not provided

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[msukkari]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

This PR modifies authentication defaults for GitHub and GitLab client factories, changing cloud service detection to default to true when no custom URL is provided, and conditionally includes authentication tokens only when present. Includes a CHANGELOG entry documenting an authentication fix and logger method corrections.

Changes

Cohort / File(s)	Summary
Documentation CHANGELOG.md	Added entry under Unreleased/Fixed describing authentication error fix for syncing public gitlab.com projects with no token
GitHub Client Factory packages/backend/src/github.ts	Changed isGitHubCloud default from false to true when no URL is provided in createOctokitFromToken
GitLab Client Factories packages/backend/src/gitlab.ts	Changed isGitLabCloud default from false to true when no URL is provided in both createGitLabFromPersonalAccessToken and createGitLabFromOAuthToken; conditionally include token/oauthToken in Gitlab constructor only if present; updated logger method calls from logger.warning to logger.warn
Repository Permission Syncer packages/backend/src/ee/repoPermissionSyncer.ts	Changed GitHub cloud detection to default isGitHubCloud to true when credentials.hostUrl is absent

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• fix(worker): Fix user driven permission syncing for self-hosted code hosts #729: Modifies cloud detection logic in code-host client factories (createOctokitFromToken, createGitLabFromOAuthToken) with URL-based determination patterns similar to this PR's changes.
• feat(ee): GitLab permission syncing #585: Updates GitLab and GitHub client-creation logic, directly overlapping with the client factory modifications in this PR.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main fix: addressing authentication errors when syncing public GitLab projects without a token, which aligns with the code changes (undefined token checks and isGitLabCloud defaults).
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[msukkari]

• undefined check for token when initializing gitlab api object
• fixes incorrect logic for isGitlabCloud and isGitHubCloud
• fixes minor bugs for log statements

Summary by CodeRabbit

Bug Fixes

• Fixed authentication errors when attempting to sync public projects without a configured authentication token
• Improved default authentication handling for public cloud services when custom configurations are not provided

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[msukkari]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

This PR modifies authentication defaults for GitHub and GitLab client factories, changing cloud service detection to default to true when no custom URL is provided, and conditionally includes authentication tokens only when present. Includes a CHANGELOG entry documenting an authentication fix and logger method corrections.

Changes

Cohort / File(s)	Summary
Documentation CHANGELOG.md	Added entry under Unreleased/Fixed describing authentication error fix for syncing public gitlab.com projects with no token
GitHub Client Factory packages/backend/src/github.ts	Changed isGitHubCloud default from false to true when no URL is provided in createOctokitFromToken
GitLab Client Factories packages/backend/src/gitlab.ts	Changed isGitLabCloud default from false to true when no URL is provided in both createGitLabFromPersonalAccessToken and createGitLabFromOAuthToken; conditionally include token/oauthToken in Gitlab constructor only if present; updated logger method calls from logger.warning to logger.warn
Repository Permission Syncer packages/backend/src/ee/repoPermissionSyncer.ts	Changed GitHub cloud detection to default isGitHubCloud to true when credentials.hostUrl is absent

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• fix(worker): Fix user driven permission syncing for self-hosted code hosts #729: Modifies cloud detection logic in code-host client factories (createOctokitFromToken, createGitLabFromOAuthToken) with URL-based determination patterns similar to this PR's changes.
• feat(ee): GitLab permission syncing #585: Updates GitLab and GitHub client-creation logic, directly overlapping with the client factory modifications in this PR.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main fix: addressing authentication errors when syncing public GitLab projects without a token, which aligns with the code changes (undefined token checks and isGitLabCloud defaults).
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[msukkari]

• undefined check for token when initializing gitlab api object
• fixes incorrect logic for isGitlabCloud and isGitHubCloud
• fixes minor bugs for log statements

Summary by CodeRabbit

Bug Fixes

• Fixed authentication errors when attempting to sync public projects without a configured authentication token
• Improved default authentication handling for public cloud services when custom configurations are not provided

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[msukkari]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

This PR modifies authentication defaults for GitHub and GitLab client factories, changing cloud service detection to default to true when no custom URL is provided, and conditionally includes authentication tokens only when present. Includes a CHANGELOG entry documenting an authentication fix and logger method corrections.

Changes

Cohort / File(s)	Summary
Documentation CHANGELOG.md	Added entry under Unreleased/Fixed describing authentication error fix for syncing public gitlab.com projects with no token
GitHub Client Factory packages/backend/src/github.ts	Changed isGitHubCloud default from false to true when no URL is provided in createOctokitFromToken
GitLab Client Factories packages/backend/src/gitlab.ts	Changed isGitLabCloud default from false to true when no URL is provided in both createGitLabFromPersonalAccessToken and createGitLabFromOAuthToken; conditionally include token/oauthToken in Gitlab constructor only if present; updated logger method calls from logger.warning to logger.warn
Repository Permission Syncer packages/backend/src/ee/repoPermissionSyncer.ts	Changed GitHub cloud detection to default isGitHubCloud to true when credentials.hostUrl is absent

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• fix(worker): Fix user driven permission syncing for self-hosted code hosts #729: Modifies cloud detection logic in code-host client factories (createOctokitFromToken, createGitLabFromOAuthToken) with URL-based determination patterns similar to this PR's changes.
• feat(ee): GitLab permission syncing #585: Updates GitLab and GitHub client-creation logic, directly overlapping with the client factory modifications in this PR.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main fix: addressing authentication errors when syncing public GitLab projects without a token, which aligns with the code changes (undefined token checks and isGitLabCloud defaults).
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[msukkari]

• undefined check for token when initializing gitlab api object
• fixes incorrect logic for isGitlabCloud and isGitHubCloud
• fixes minor bugs for log statements

Summary by CodeRabbit

Bug Fixes

• Fixed authentication errors when attempting to sync public projects without a configured authentication token
• Improved default authentication handling for public cloud services when custom configurations are not provided

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[msukkari]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

This PR modifies authentication defaults for GitHub and GitLab client factories, changing cloud service detection to default to true when no custom URL is provided, and conditionally includes authentication tokens only when present. Includes a CHANGELOG entry documenting an authentication fix and logger method corrections.

Changes

Cohort / File(s)	Summary
Documentation CHANGELOG.md	Added entry under Unreleased/Fixed describing authentication error fix for syncing public gitlab.com projects with no token
GitHub Client Factory packages/backend/src/github.ts	Changed isGitHubCloud default from false to true when no URL is provided in createOctokitFromToken
GitLab Client Factories packages/backend/src/gitlab.ts	Changed isGitLabCloud default from false to true when no URL is provided in both createGitLabFromPersonalAccessToken and createGitLabFromOAuthToken; conditionally include token/oauthToken in Gitlab constructor only if present; updated logger method calls from logger.warning to logger.warn
Repository Permission Syncer packages/backend/src/ee/repoPermissionSyncer.ts	Changed GitHub cloud detection to default isGitHubCloud to true when credentials.hostUrl is absent

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• fix(worker): Fix user driven permission syncing for self-hosted code hosts #729: Modifies cloud detection logic in code-host client factories (createOctokitFromToken, createGitLabFromOAuthToken) with URL-based determination patterns similar to this PR's changes.
• feat(ee): GitLab permission syncing #585: Updates GitLab and GitHub client-creation logic, directly overlapping with the client factory modifications in this PR.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main fix: addressing authentication errors when syncing public GitLab projects without a token, which aligns with the code changes (undefined token checks and isGitLabCloud defaults).
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}