summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Use = separator for stream offset STOMPstomp-for-streamsArnaud Cogoluègnes2021-04-262-8/+4
| | | | | | | | | For offset (e.g. offset=5000) and timestamp (timetamp=5000). It was ":" before, but this is already the separator for key/value in STOMP headers. This was actually working with the client used to test, but other clients may get confused. References #3002.
* Support streams in STOMP pluginArnaud Cogoluègnes2021-04-265-2/+52
| | | | | | | | | | This commit introduces the support of an x-stream-offset header in the SUBSCRIBE frame to start consuming from a specific place in a stream. The possible values are first, last, next, offset:<offset-value> (e.g. offset:40000), timestamp:<timestamp-in-seconds> (e.g. timestamp:1619428685). This commit also propagates the x-stream-offset header in the MESSAGE frame to know the offset of a the delivered message in the stream.
* Merge pull request #2993 from rabbitmq/rebalance-ignore-drainedMichael Klishin2021-04-231-1/+1
|\ | | | | Ignore drained nodes when rebalancing
| * Revert "Increase ra:members/1 timeout to 60s"rebalance-ignore-drainedMichal Kuratczyk2021-04-221-2/+1
| | | | | | | | | | | | It did not solve the problem and made the failure slower. This reverts commit 510230302e0adf657b3f6f7c849962ae9796ca2a.
| * Merge pull request #2973 from luos/mirror-sync-reduce-memory-useMichael Klishin2021-04-222-3/+15
| | | | | | | | | | | | Classic queue mirrors should also continue pushing messages to disk when syncing (cherry picked from commit a728662fd79e00b1c1713111d2fcfe5a9113753e)
| * Set a default for consumer_timeoutkjnilsson2021-04-222-0/+9
| | | | | | | | | | So that faulty consumers that will never ack a pending messages have their channels closed after 15 minutes.
| * Change consistent to a dirty readMichal Kuratczyk2021-04-211-1/+1
| | | | | | | | | | | | This is executed once per queue so potentially many, many, times and a dirty read is much faster. The state of maintenance mode on the nodes is unlikely to change during rebalancing.
| * Increase ra:members/1 timeout to 60sdcorbacho2021-04-211-1/+2
| | | | | | | | | | Transfer leadership is a slow command, it's acceptable to wait until the ra cluster is ready to answer
| * Ignore drained nodes when rebalancingMichal Kuratczyk2021-04-211-1/+1
| | | | | | | | When running `rabbitmq-queues rebalance`, drained nodes (nodes in maintenance mode) should not recive QQ leaders nor CMQ masters.
* | Merge pull request #2998 from rabbitmq/tag-branchMichael Klishin2021-04-232-4/+11
|\ \ | | | | | | Tag docker image with branch name
| * | Tag docker image with branch nameDavid Ansari2021-04-232-4/+11
|/ / | | | | | | | | | | | | | | | | in addition to tagging the docker image with the Git commit SHA. This enables the cluster-operator to run system-tests against latest rabbitmq-server master branch which allows us to fix any breaking changes early and to test unreleased features such as rabbitmq_stream plugin.
* | Merge pull request #2973 from luos/mirror-sync-reduce-memory-useMichael Klishin2021-04-222-3/+15
| | | | | | | | | | | | Classic queue mirrors should also continue pushing messages to disk when syncing (cherry picked from commit a728662fd79e00b1c1713111d2fcfe5a9113753e)
* | Merge pull request #2990 from rabbitmq/consumer-timeout-defaultMichael Klishin2021-04-212-0/+9
|\ \ | |/ |/| Set a default for consumer_timeout
| * Set a default for consumer_timeoutconsumer-timeout-defaultkjnilsson2021-04-212-0/+9
| | | | | | | | | | So that faulty consumers that will never ack a pending messages have their channels closed after 15 minutes.
* | Merge pull request #2989 from Ayanda-D/channel-interceptor-amqp-errorsMichael Klishin2021-04-203-4/+69
|\ \ | | | | | | Allow AMQP error responses in channel interceptors
| * | Allow #amqp_error{} responses in channel interceptorsAyanda-D2021-04-203-4/+69
|/ /
* | Merge pull request #2985 from rabbitmq/mk-decode-other-name-as-directory-stringMichael Klishin2021-04-201-4/+6
|\ \ | |/ |/| Mk decode other name as directory string
| * Allow for more ASN.1 string (sub)typesMichael Klishin2021-04-201-2/+2
| |
| * Decode other name using 'OTP-PUB-KEY':decode/2Michael Klishin2021-04-201-3/+5
| | | | | | | | | | | | | | | | and assume it is a string-like value ("directory string") because other values would not make much sense in the username extraction context. References #2983.
| * SAN of type otherName: strip two leading charactersMichael Klishin2021-04-201-1/+1
|/ | | | | | | | | | | | instead of specific ones since they will vary with the payload (one of them likely indicates UTF string length). This is still not perfect because we limit the maximum allowed length but it works fine with identifiers up to 100 characters long, which should be good enough for this best effort handling of an abscure SAN type. References ##2983.
* Add rabbitmq_peer_discovery_k8s tests to bazelPhilip Kuryloski2021-04-191-0/+83
|
* Add rabbitmq_peer_discovery_etcd tests to bazelPhilip Kuryloski2021-04-192-0/+120
|
* Drop some Erlang 21-specific codeMichael Klishin2021-04-191-23/+0
| | | | We no longer support it even in v3.8.x.
* Pass DialyzerMichael Klishin2021-04-192-1/+7
| | | | | x.509 certificate extensions do not really have types in OTP's public_key, so our hands are tied.
* rabbit_common: pass DialyzerMichael Klishin2021-04-191-1/+1
|
* Merge branch 'fix-sh-concatenated-literal-parsing'Jean-Sébastien Pédron2021-04-192-6/+148
|\
| * rabbit_env: Fix Bourne shell concatenated literal parsingJean-Sébastien Pédron2021-04-192-6/+148
|/ | | | | | | | | | | | | The parser didn't handle literals of the form: 'single-quoted'unquoted'single-quoted-again'"or-even-double-quoted" In particular, the unquoted parsing assumed that nothing else could follow it. The testsuite is extended with the issue reporter's case. While here, improve escaped characters handling. They used to be not parsed specifically at all. Fixes #2969.
* Add rabbitmq_peer_discovery_common tests to bazelPhilip Kuryloski2021-04-192-23/+95
|
* Remove a leftover commentMichael Klishin2021-04-191-3/+0
|
* Test rabbitmq_mqtt with bazelPhilip Kuryloski2021-04-193-4/+399
|
* Merge pull request #2984 from rabbitmq/rabbitmq-server-2983Michael Klishin2021-04-193-8/+31
|\ | | | | x.509 certificate auth mechanism: support (some) SANS of type otherName
| * x.509 certificate auth mechanism: support (some) SANS of type otherNameMichael Klishin2021-04-193-8/+31
| | | | | | | | | | | | | | | | | | | | | | | | Note that the type by definition contains arbitrary values. According to the OTP types, they are triplets that represent effectively a key/value pair. So we assume the pair is a string that needs a bit massaging, namely stripping the UTF encoding prefix OTP AnotherName decoder leaves in. Kudos to @Thibi2000 for providing an example value. Closes #2983.
* | Update README.mdMichael Klishin2021-04-191-0/+10
|/
* Squash a warning that breaks CI compilationMichael Klishin2021-04-161-2/+0
|
* Remove stream plugin instructionsArnaud Cogoluègnes2021-04-161-16/+1
| | | No longer valid, use the usual process to run a plugin.
* Merge pull request #2982 from rabbitmq/mk-correctly-parse-tag-listsMichael Klishin2021-04-162-13/+38
|\ | | | | Correctly parse tags provided as a list
| * CosmeticsMichael Klishin2021-04-161-3/+3
| |
| * Correct parse tags provided as a listMichael Klishin2021-04-162-13/+38
| | | | | | | | Discovered while testing a PR for rabbit-hole
* | Merge pull request #2980 from skapin/patch-1Michael Klishin2021-04-161-4/+18
|\ \ | |/ |/| Set correct field for extra scope and improve doc
| * Set correct field for extra scope and improve docSkapiN2021-04-161-4/+18
|/ | | | | - Replace additional_rabbitmq_scopes in doc for the correct extra_scopes_source value - Add samples - More details for audiance checkup on token
* rabbit_fifo: fix failing testskjnilsson2021-04-162-5/+10
| | | | That regressed after refactoring how delivery effects are emitted.
* Mark feature_flags_SUITE case flaky in bazelPhilip Kuryloski2021-04-161-0/+1
|
* buildifierPhilip Kuryloski2021-04-162-4/+4
|
* Refactor and dedupe .bazel file contentPhilip Kuryloski2021-04-1617-332/+90
|
* Make a smaller subset of the rabbit feature_flags_SUITE "exclusive"Philip Kuryloski2021-04-161-4/+28
| | | | This should alleviate lack of disk space in github actons
* Merge pull request #2976 from rabbitmq/qq-in-memory-limit-bug-fixMichael Klishin2021-04-153-38/+86
|\ | | | | QQ: ensure that messages are delivered in order
| * QQ: ensure that messages are delivered in orderqq-in-memory-limit-bug-fixkjnilsson2021-04-153-38/+86
| | | | | | | | | | | | | | | | | | | | | | In the case where there are some messages kept in memory mixed with some that are not it is possible that a messages are delivered to the consuming channel with gaps/out of order which would in some cases cause the channel to treat them as re-sends it has already seen and just discard them. When this happens the messages get stuck in the consumer state inside the queue and are never seen by the client consumer and thus never acked. When this happen the release cursors can't be emitted as the smallest raft index will be one of the stuck messages.
* | Mark rabbit_stream_queue_SUITE-cluster_size_3_1 as flakyPhilip Kuryloski2021-04-151-12/+15
| |
* | Add rabbitmq_management_agent suites to bazelPhilip Kuryloski2021-04-151-15/+152
| |
* | Add rabbitmq_management tests to bazelPhilip Kuryloski2021-04-151-2/+318
| |