Skip to content

[v16] display all security group rules for a port range#47077

Merged
GavinFrazar merged 1 commit intobranch/v16from
gavinfrazar/v16-display-all-security-group-rules-per-port
Oct 2, 2024
Merged

[v16] display all security group rules for a port range#47077
GavinFrazar merged 1 commit intobranch/v16from
gavinfrazar/v16-display-all-security-group-rules-per-port

Conversation

@GavinFrazar
Copy link
Copy Markdown
Contributor

Changelog: Fixed a bug where only one IP CIDR block security group rule for a port range was displayed in the web UI RDS enrollment wizard when viewing a security group.

Backport #47009 to branch/v16.

there was a straightforward merge conflict with a <Text> on v16 vs <H2> in master - I kept the content of the html tag the same as the backported PR but left it as a <Text>. Relevant snippet was:

        <Text mb={4} typography="h4">
          {ruleType === 'inbound' ? 'Inbound' : 'Outbound'} Rules for [{name}]
        </Text>

@GavinFrazar GavinFrazar added ui backport discover Issues related to Teleport Discover labels Oct 2, 2024
@github-actions github-actions Bot requested a review from avatus October 2, 2024 02:15
@github-actions github-actions Bot requested a review from kimlisa October 2, 2024 02:15
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from avatus October 2, 2024 18:46
@GavinFrazar GavinFrazar added this pull request to the merge queue Oct 2, 2024
Merged via the queue into branch/v16 with commit c8af2a7 Oct 2, 2024
@GavinFrazar GavinFrazar deleted the gavinfrazar/v16-display-all-security-group-rules-per-port branch October 2, 2024 21:04
@camscale camscale mentioned this pull request Oct 16, 2024
@fheinecke fheinecke mentioned this pull request Apr 9, 2025
@fheinecke fheinecke mentioned this pull request Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport discover Issues related to Teleport Discover size/sm ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants