Skip to content
Snippets Groups Projects

Add support for custom domains to the Pages internal API

Merged Krasimir Angelov requested to merge 61927-pages-custom-domain-virtual-domain into master
All threads resolved!

What does this MR do?

This is another extraction from https://gitlab.com/gitlab-org/gitlab-ce/issues/61927 in order break the later in smaller chunks.

This MR add support for custom pages domains to the endpoint introduced in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/32732. Namespaces domains are intentionally left out cause this is where the data migration and tricky queries will happen, this way we can introduce most of the code we'll eventually need before we add the last piece.

Keep in mind this branch is based on https://gitlab.com/gitlab-org/gitlab-ce/tree/61927-skeleton-pages-internal-api and can be merged only once https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/32732 is merged.

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Performance 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

Related #61927 (moved)

Edited by Kamil Trzciński

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    • f7c4e615 - Add support for custom domains to the Pages internal API

    Compare with previous version

  • Krasimir Angelov added 204 commits

    added 204 commits

    Compare with previous version

  • Krasimir Angelov changed the description

    changed the description

  • Krasimir Angelov changed target branch from 61927-skeleton-pages-internal-api to master

    changed target branch from 61927-skeleton-pages-internal-api to master

  • added 1 commit

    • 676675dc - Add support for custom domains to the internal Pages API

    Compare with previous version

  • Krasimir Angelov resolved all threads

    resolved all threads

  • assigned to @ayufan

  • Kamil Trzciński
  • Kamil Trzciński
  • Kamil Trzciński
  • Kamil Trzciński
  • Krasimir Angelov added 3 commits

    added 3 commits

    • 268c698f - Add tests for Pages::VirtualDomain
    • 5809400d - Add tests for PagesDomain#pages_virtual_domain
    • 91af0cf4 - Add tests for Project#pages_lookup_path

    Compare with previous version

  • added 1 commit

    • 5374f423 - Add tests for ProjectFeature's #public_pages? and #private_pages?

    Compare with previous version

  • assigned to @ayufan

  • Kamil Trzciński approved this merge request

    approved this merge request

  • Kamil Trzciński resolved all threads

    resolved all threads

  • @krasio lets keep this :train: going :)

  • Kamil Trzciński marked the checklist item Code review guidelines as completed

    marked the checklist item Code review guidelines as completed

  • Kamil Trzciński marked the checklist item Merge request performance guidelines as completed

    marked the checklist item Merge request performance guidelines as completed

  • Kamil Trzciński marked the checklist item Style guides as completed

    marked the checklist item Style guides as completed

  • Kamil Trzciński marked the checklist item Separation of EE specific content as completed

    marked the checklist item Separation of EE specific content as completed

  • mentioned in commit 68e8bbdb

  • Krasimir Angelov mentioned in merge request !32945 (closed)

    mentioned in merge request !32945 (closed)

  • Please register or sign in to reply
    Loading