Skip to content

Remove unnecessary header X-Iceberg-Access-Delegation for Ceph#3744

Merged
MonkeyCanCode merged 2 commits intoapache:mainfrom
MonkeyCanCode:ceph_getting_start_cleanup
Feb 12, 2026
Merged

Remove unnecessary header X-Iceberg-Access-Delegation for Ceph#3744
MonkeyCanCode merged 2 commits intoapache:mainfrom
MonkeyCanCode:ceph_getting_start_cleanup

Conversation

@MonkeyCanCode
Copy link
Contributor

Same discussed in #3591, having empty value for X-Iceberg-Access-Delegation is kind misleading and we don't need this for the getting start example to work.

Checklist

  • 🛡️ Don't disclose security issues! (contact [email protected])
  • 🔗 Clearly explained why the changes are needed, or linked related issues: Fixes #
  • 🧪 Added/updated tests with good coverage, or manually tested (and explained how)
  • 💡 Added comments for complex logic
  • 🧾 Updated CHANGELOG.md (if needed)
  • 📚 Updated documentation in site/content/in-dev/unreleased (if needed)

@github-project-automation github-project-automation bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board Feb 12, 2026
@MonkeyCanCode MonkeyCanCode merged commit 9b6b805 into apache:main Feb 12, 2026
15 checks passed
@github-project-automation github-project-automation bot moved this from Ready to merge to Done in Basic Kanban Board Feb 12, 2026
Copy link
Contributor

@dimas-b dimas-b left a comment

Choose a reason for hiding this comment

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

+1

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants