Skip to content

[bugfix] Fix config error when authorization_header is not set in the httpjson handler - #3711

Open
ekouts wants to merge 1 commit into
reframe-hpc:masterfrom
ekouts:bugfix/auth_header
Open

[bugfix] Fix config error when authorization_header is not set in the httpjson handler#3711
ekouts wants to merge 1 commit into
reframe-hpc:masterfrom
ekouts:bugfix/auth_header

Conversation

@ekouts

@ekouts ekouts commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Reframe would raise a config error when authorization_header is not set in the httpjson handler.

ERROR: failed to load configuration: httpjson: 'authorization_header' has the wrong signature: it must be 'authorization_header()'

@ekouts ekouts added this to the ReFrame 4.10.3 milestone Aug 24, 2026
@ekouts
ekouts requested review from gppezzi and vkarak August 24, 2026 11:52
@ekouts ekouts self-assigned this Aug 24, 2026
@ekouts ekouts changed the title [bugfix] Fix http config error when auth header is not set [bugfix] Fix http config error when authorization_header is not set in the httpjson handler Aug 24, 2026
@ekouts ekouts changed the title [bugfix] Fix http config error when authorization_header is not set in the httpjson handler [bugfix] Fix config error when authorization_header is not set in the httpjson handler Aug 24, 2026
@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.90%. Comparing base (307a37c) to head (1cfbe2f).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3711   +/-   ##
=======================================
  Coverage   91.90%   91.90%           
=======================================
  Files          62       62           
  Lines       13782    13782           
=======================================
  Hits        12667    12667           
  Misses       1115     1115           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Eirini Koutsaniti <ekoutsaniti@gmail.com>
@ekouts
ekouts force-pushed the bugfix/auth_header branch from 60e590b to 1cfbe2f Compare August 24, 2026 11:58
@vkarak

vkarak commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Can you describe more precisely when it fails? With a config snippet maybe? Afaiu, the authorization_header breaks existing httpsjson handler definitions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants