Merge ~jessica-youjeong/ubuntu/+source/librabbitmq:031822-focal-devel-librabbitmq into ubuntu/+source/librabbitmq:ubuntu/focal-devel

Proposed by Jess Jang
Status: Work in progress
Proposed branch: ~jessica-youjeong/ubuntu/+source/librabbitmq:031822-focal-devel-librabbitmq
Merge into: ubuntu/+source/librabbitmq:ubuntu/focal-devel
Diff against target: 87 lines (+65/-0)
3 files modified
debian/changelog (+21/-0)
debian/patches/series (+2/-0)
debian/patches/server_flag_deprecation.patch (+42/-0)
Reviewer Review Type Date Requested Status
Utkarsh Gupta Pending
Review via email: mp+417112@code.launchpad.net

Description of the change

This MP contains a fix for LP: 1888313, which I intend to SRU as well, so this is a fix for Focal which is backported from link below. The package was successfully built using sbuild and I also ran the autopkgtest and all the tests are passing.

Requesting you to please review and sponsor the upload. Thank you!

[https://code.launchpad.net/~jessica-youjeong/ubuntu/+source/librabbitmq/+git/librabbitmq/+merge/417036]

To post a comment you must log in.

Unmerged commits

a04b6e1... by Jess Jang

changelog

a54540f... by Jess Jang

  * d/p/server-flag-deprecation.patch:
    add patch to fix issue parsing hostname in --server flag.
    (LP: #1888313)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 9378ba2..f397bd1 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,24 @@
6+librabbitmq (0.10.0-1ubuntu1.20.04.1) focal; urgency=medium
7+
8+ * d/p/server-flag-deprecation.patch:
9+ add patch to fix issue parsing hostname in --server flag.
10+ (LP: #1888313)
11+
12+ -- Jess Jang <jess.jang@canonical.com> Thu, 03 Feb 2022 14:30:36 -0600
13+
14+librabbitmq (0.10.0-1ubuntu1) jammy; urgency=medium
15+
16+ * d/p/ssl-remove-reference-to-FIPS_set_mode.patch: backported
17+ to fix the build against OpenSSL 3.0 (LP: #1945812)
18+
19+ -- Simon Chopin <simon.chopin@canonical.com> Wed, 17 Nov 2021 14:22:23 +0100
20+
21+librabbitmq (0.10.0-1build1) impish; urgency=medium
22+
23+ * No-change rebuild to build packages with zstd compression.
24+
25+ -- Matthias Klose <doko@ubuntu.com> Thu, 07 Oct 2021 12:17:49 +0200
26+
27 librabbitmq (0.10.0-1) unstable; urgency=high
28
29 * New upstream release, fixing CVE-2019-18609 (Closes: #946005).
30diff --git a/debian/patches/series b/debian/patches/series
31index f96d27b..3365953 100644
32--- a/debian/patches/series
33+++ b/debian/patches/series
34@@ -1,3 +1,5 @@
35 cmake_with_ssl.patch
36 use_cmake_package.patch
37 disable-test-basic.patch
38+ssl-remove-reference-to-FIPS_set_mode.patch
39+server_flag_deprecation.patch
40diff --git a/debian/patches/server_flag_deprecation.patch b/debian/patches/server_flag_deprecation.patch
41new file mode 100644
42index 0000000..de99b0b
43--- /dev/null
44+++ b/debian/patches/server_flag_deprecation.patch
45@@ -0,0 +1,42 @@
46+From 637f0275ffcc2bfb58c94f0bf6e290aa40c1695a Mon Sep 17 00:00:00 2001
47+From: Alan Antonuk <alan.antonuk@gmail.com>
48+Date: Fri, 7 Aug 2020 05:07:50 +0000
49+Subject: [PATCH] tool: set port when not specified in --server flag
50+
51+Set the port to 5672 and 5671 in SSL mode in the tools when the --server
52+flag is used, but does not specify a port.
53+
54+From 54d00bee5cc4603dc96af5f86c899210689dfeeb Mon Sep 17 00:00:00 2001
55+From: Alan Antonuk <alan.antonuk@gmail.com>
56+Date: Sun, 9 Aug 2020 06:44:28 +0000
57+Subject: [PATCH] tool: fix issue parsing hostname in --server flag
58+
59+Set both the host and port when a ':' isn't found in in the --server
60+flag. The attempted fix in #622 was not complete as the host was not
61+properly set.
62+
63+Fixes: #621
64+
65+Origin: upstream;
66+ https://github.com/alanxz/rabbitmq-c/commit/637f0275ffcc2bfb58c94f0bf6e290aa40c1695a
67+ https://github.com/alanxz/rabbitmq-c/commit/4d03c849868231dc217785c6f0e78eab9cd79fd5
68+Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/librabbitmq/+bug/1888313
69+Last-Updated: 2022-01-19
70+
71+--- a/tools/common.c
72++++ b/tools/common.c
73+@@ -240,6 +240,14 @@
74+ if (ci->port < 0 || ci->port > 65535 || port_end == colon + 1 ||
75+ *port_end != 0)
76+ die("bad server port number in '%s'", amqp_server);
77++ } else {
78++ ci->host = amqp_server;
79++ ci->port = 5672;
80++#if WITH_SSL
81++ if (amqp_ssl) {
82++ ci->port = 5671;
83++ }
84++#endif
85+ }
86+
87+ #if WITH_SSL

Subscribers

People subscribed via source and target branches