New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix local port regex in listen_ports_facts #4092
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution! Can you please add a changelog fragment? Thanks!
Thsi PR fix the bug reported in ansible-collections#4091
Sure, done |
changelogs/fragments/4092-fix_local_ports_regex_listen_ports_facts.yaml
Outdated
Show resolved
Hide resolved
…acts.yaml Co-authored-by: Felix Fontein <felix@fontein.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks formally good to me. Would be glad if someone else could check / test this.
CC @moonrail since you added |
I did some very basic testing, and it looks correct to me. If nobody complains by tomorrow evening (i.e. in roughly 24 hours), I'll merge. |
Backport to stable-3: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 6a7811f on top of patchback/backports/stable-3/6a7811f6963ad1f5f92342b786e70a809e1c9e08/pr-4092 Backporting merged PR #4092 into main
🤖 @patchback |
@gaetan-craft thanks for fixing this! |
Backport to stable-4: 💚 backport PR created✅ Backport PR branch: Backported as #4128 🤖 @patchback |
* Fix local port regex Thsi PR fix the bug reported in #4091 * Update changelogs/fragments/4092-fix_local_ports_regex_listen_ports_facts.yaml Co-authored-by: Felix Fontein <felix@fontein.de> Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit 6a7811f) Co-authored-by: gaetan-craft <97035736+gaetan-craft@users.noreply.github.com>
SUMMARY
This PR fix the bug reported in #4091 by fixing the regex used to parse ss local connection entry
ISSUE TYPE
COMPONENT NAME
listen_ports_facts