Skip to content

change: [DPS-35843] - Create destination, validate if host and bucket name match#13176

Merged
mduda-akamai merged 6 commits intolinode:developfrom
kagora-akamai:feature/DPS-35843-create-destination-validate-host-and-bucket-name-match
Dec 17, 2025
Merged

change: [DPS-35843] - Create destination, validate if host and bucket name match#13176
mduda-akamai merged 6 commits intolinode:developfrom
kagora-akamai:feature/DPS-35843-create-destination-validate-host-and-bucket-name-match

Conversation

@kagora-akamai
Copy link
Contributor

@kagora-akamai kagora-akamai commented Dec 8, 2025

Description 📝

In create destination form, add validation for Host and Bucket Name fields.
Bucket name must be equal to first part of the host name

Changes 🔄

  • match validation on bucket name field
  • match validation on host field

Scope 🚢

Upon production release, changes in this PR will be visible to:

  • Some customers (e.g. in Beta or Limited Availability)

Target release date 🗓️

December 2026

Preview 📷

Before After
Screenshot 2025-12-08 at 14 49 43 Screenshot 2025-12-15 at 15 11 50

How to test 🧪

Prerequisites

start manager

Verification steps

  • navigate to logs/delivery/destinations/create
  • fill Host field with value {bucket}.{rest of the host name}
  • fill bucket name field with value {bucket}
  • modify {bucket} value in any of the fields, click test connection, or validate fields on blur to see error messages
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
📱 Providing mobile support


  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@kagora-akamai kagora-akamai marked this pull request as ready for review December 8, 2025 14:02
@kagora-akamai kagora-akamai requested a review from a team as a code owner December 8, 2025 14:02
@kagora-akamai kagora-akamai marked this pull request as draft December 8, 2025 14:33
@kagora-akamai kagora-akamai marked this pull request as ready for review December 8, 2025 18:21
@kagora-akamai kagora-akamai requested a review from a team as a code owner December 8, 2025 18:21
@kagora-akamai kagora-akamai requested review from jdamore-linode and removed request for a team December 8, 2025 18:21
Copy link
Contributor

@dwiley-akamai dwiley-akamai left a comment

Choose a reason for hiding this comment

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

Code review ✅
Verification steps ✅

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Dec 15, 2025
@kagora-akamai kagora-akamai added the Approved Multiple approvals and ready to merge! label Dec 16, 2025
@mduda-akamai mduda-akamai merged commit 3e1af13 into linode:develop Dec 17, 2025
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Dec 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge! Logs

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants