Skip to content
Snippets Groups Projects

Support job token auth for read-only changelog generation API

Merged Timo Furrer requested to merge get-changelog-api-job-token into master
All threads resolved!

What does this MR do and why?

Support job token auth for read-only changelog generation API

This change set adds support that allows authentication via CI_JOB_TOKEN for the read-only (GET) Changelog generation API. This is super useful to generate a changelog or release notes without any additional tokens.

Since this is a read-only endpoint the security impact is very low.

Changelog: added

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
  • A deleted user added backend documentation labels

    added backend documentation labels

  • Timo Furrer requested review from @vyaklushin

    requested review from @vyaklushin

  • Vasilii Iakliushin approved this merge request

    approved this merge request

  • Vasilii Iakliushin requested review from @cmaxim

    requested review from @cmaxim

  • Before you set this MR to auto-merge

    This merge request will progress on pipeline tiers until it reaches the last tier: pipelinetier-3. We will trigger a new pipeline for each transition to a higher tier.

    Before you set this MR to auto-merge, please check the following:

    • You are the last maintainer of this merge request
    • The latest pipeline for this merge request is pipelinetier-3 (You can find which tier it is in the pipeline name)
    • This pipeline is recent enough (created in the last 8 hours)

    If all the criteria above apply, please set auto-merge for this merge request.

    See pipeline tiers and merging a merge request for more details.

  • E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for bff56b1c

    expand test summary
    +-------------------------------------------------------------+
    |                       suites summary                        |
    +--------+--------+--------+---------+-------+-------+--------+
    |        | passed | failed | skipped | flaky | total | result |
    +--------+--------+--------+---------+-------+-------+--------+
    | Plan   | 1      | 0      | 0       | 0     | 1     | ✅     |
    | Govern | 1      | 0      | 0       | 0     | 1     | ✅     |
    | Create | 2      | 0      | 0       | 0     | 2     | ✅     |
    +--------+--------+--------+---------+-------+-------+--------+
    | Total  | 4      | 0      | 0       | 0     | 4     | ✅     |
    +--------+--------+--------+---------+-------+-------+--------+

    e2e-test-on-cng: :white_check_mark: test report for bff56b1c

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Package     | 24     | 0      | 14      | 0     | 38    | ✅     |
    | Plan        | 86     | 0      | 8       | 0     | 94    | ✅     |
    | Analytics   | 2      | 0      | 0       | 1     | 2     | ✅     |
    | Monitor     | 8      | 0      | 12      | 0     | 20    | ✅     |
    | Create      | 140    | 0      | 20      | 0     | 160   | ✅     |
    | Secure      | 2      | 0      | 5       | 0     | 7     | ✅     |
    | Data Stores | 33     | 0      | 10      | 0     | 43    | ✅     |
    | Verify      | 49     | 0      | 16      | 0     | 65    | ✅     |
    | Govern      | 84     | 0      | 9       | 1     | 93    | ✅     |
    | Release     | 5      | 0      | 1       | 0     | 6     | ✅     |
    | Manage      | 1      | 0      | 9       | 0     | 10    | ✅     |
    | Configure   | 0      | 0      | 3       | 0     | 3     | ➖     |
    | ModelOps    | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Fulfillment | 2      | 0      | 7       | 1     | 9     | ✅     |
    | Ai-powered  | 0      | 0      | 2       | 0     | 2     | ➖     |
    | Growth      | 0      | 0      | 2       | 0     | 2     | ➖     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 436    | 0      | 119     | 3     | 555   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
  • requested review from @brendan777

  • Timo Furrer added 1 commit

    added 1 commit

    • bff56b1c - Apply 2 suggestion(s) to 2 file(s)

    Compare with previous version

  • Timo Furrer requested review from @brendan777

    requested review from @brendan777

  • Ghost User
  • Brendan Lynch approved this merge request

    approved this merge request

  • Brendan Lynch resolved all threads

    resolved all threads

  • Author Maintainer

    We seem to have all approvals - so let me go ahead an kick off the merge, thanks everyone for the reviews :handshake:

    EDIT: we still need a backend maintainer review :grinning:

    @seanarnold would you do us the honor? :pray:

    Edited by Timo Furrer
  • Timo Furrer requested review from @seanarnold

    requested review from @seanarnold

  • Sean Arnold approved this merge request

    approved this merge request

  • Sean Arnold enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • merged

  • Sean Arnold mentioned in commit 728cd704

    mentioned in commit 728cd704

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading