gogrlx/nats-server
1
0
Fork 0
mirror of https://github.com/gogrlx/nats-server.git synced 2026-04-14 10:10:42 -07:00
Code Issues Packages Projects Releases Wiki Activity
6,590 Commits 30 Branches 48 Tags
8e64fa5f3790df82aa06ecfb2e2cf33fbfcbd638
Commit Graph

4412 Commits

Author SHA1 Message Date
Derek Collison
8e64fa5f37 Merge branch 'main' into dev 2022-11-17 06:00:24 -08:00
Raymond
4d8964e57b Added stream created timestamp to stream detail 2022-11-17 13:59:58 +01:00
Ivan Kozlovic
b2f3d1d706 Merge branch 'main' into dev 2022-11-16 17:25:08 -07:00
Ivan Kozlovic
deb9d252a3 Bump version to 2.9.7-RC2 
Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
 2022-11-16 17:23:15 -07:00
Ivan Kozlovic
74a16b0097 Merge pull request #3640 from nats-io/fix_3639 
[FIXED] JetStream: WorkQueue not preventing overlapping consumers
 2022-11-16 17:22:35 -07:00
Ivan Kozlovic
49faba9e33 [FIXED] JetStream: WorkQueue not preventing overlapping consumers 
A stream with a WorkQueue retention policy is supposed to allow
more than one consumer if they user filtered subjects, but those
subjects should not overlap.

There was an issue that if a new consumer had a filter subject
"wider" than an existing one, the error was not detected and
the new consumer was incorrectly accepted.

Resolves #3639

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
 2022-11-16 17:09:30 -07:00
Derek Collison
a76121f460 Merge branch 'main' into dev 2022-11-16 15:27:57 -08:00
Derek Collison
e4b5b7ec64 Make mirror consumers use filtered version of consumer create like source consumers. 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-16 15:12:14 -08:00
Derek Collison
eebf782006 Merge branch 'main' into dev 2022-11-16 12:12:33 -08:00
Derek Collison
6cb7f68ef7 Updates based on feedback w/ Ivan 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-16 11:53:46 -08:00
Derek Collison
aa57adbcd0 Use default here 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-16 11:31:49 -08:00
Derek Collison
f696c6559e Adjust test for jitter on ephemerals. 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-15 17:51:01 -08:00
Derek Collison
08c94096db Allow any type of activity to prolong auto cleanup of a consumer. 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-15 17:25:18 -08:00
Ivan Kozlovic
deadc05c82 Merge branch 'main' into dev 2022-11-15 17:07:09 -07:00
Colin Sullivan
9f6db71937 Fix concurrent map write 
Signed-off-by: Colin Sullivan <colin@synadia.com>
 2022-11-15 16:26:35 -07:00
Derek Collison
d98d0278a2 Bump to 2.9.7-RC1 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-15 12:13:10 -08:00
Derek Collison
4b02624d9b Merge branch 'main' into dev 2022-11-15 12:04:38 -08:00
Ivan Kozlovic
37ae002353 Merge branch 'main' into dev 2022-11-15 12:25:07 -07:00
Derek Collison
9f241f3322 Offload signaling to consumers when number is large. 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-15 11:25:07 -08:00
Ivan Kozlovic
0f79b913ec Merge pull request #3631 from nats-io/fix_3626 
[FIXED] JetStream: possible panic on stream info when leader not elected
 2022-11-15 12:24:10 -07:00
Ivan Kozlovic
6ffa6d1e4b [FIXED] JetStream: possible panic on stream info when leader not elected 
It is possible that a stream info request would be handled at a
time where the raft group would not yet be set/created, causing
a panic.

Resolves #3626 (at least the panic reports there)

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
 2022-11-15 11:56:41 -07:00
Derek Collison
c83d5ce4c6 Merge branch 'main' into dev 2022-11-15 08:01:51 -08:00
Derek Collison
b92ea86b80 When reading state, even on consumer init, should lock. 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-15 07:40:44 -08:00
Ivan Kozlovic
095b13c4be Merge branch 'main' into dev 2022-11-14 17:12:50 -07:00
Ivan Kozlovic
9e36918f41 [FIXED] JetStream: Account removal leaks subscriptions 
Resolve #3623

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
 2022-11-14 16:40:21 -07:00
Derek Collison
051b986b58 Merge branch 'main' into dev 2022-11-14 15:06:41 -08:00
Derek Collison
b6ef2c8910 Auto cleanup dangling messages from interest policy streams on server start. 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-14 15:02:22 -08:00
Derek Collison
36ef788112 When determing whether we need an ack, no need to copy since under consumer lock. 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-14 11:47:31 -08:00
Derek Collison
43a4ba6d9f Merge branch 'main' into dev 2022-11-14 08:40:58 -08:00
Derek Collison
b76b6a1f68 Merge pull request #3620 from nats-io/fix-3612 
[FIXED] Fix logic bug that would prevent some stream messages from being deleted.
 2022-11-14 08:40:13 -08:00
Waldemar Quevedo
39185c11a6 Set max payload in gofuzz test (#3622) 
* Set max payload in gofuzz test

This was causing the test to overrun mem limits due to max payload size
check being skipped in the parser when processing a `pub` protocol line.

Signed-off-by: Waldemar Quevedo <wally@nats.io>
 2022-11-14 08:28:19 -08:00
Derek Collison
47dd97e389 Fix logic bug that would prevent some messages from being deleted on an interest based stream. 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-13 17:32:38 -08:00
Derek Collison
a5398e1902 Merge pull request #3617 from nats-io/add-unique-tag 
add unique_tag to JetStreamConfig for varz/jsz visibility
 2022-11-12 09:08:53 -08:00
Todd Beets
40bbfbacc9 correct spaces to tabs in raw string literal 2022-11-12 07:53:20 -08:00
Derek Collison
a2f8dbfafc Merge branch 'main' into dev 2022-11-11 17:08:39 -08:00
Derek Collison
3ac6052b32 Updated pae threshold and reporting modulo to not spam logs as much. 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-11 16:08:58 -08:00
Ivan Kozlovic
bc084d4e27 Merge branch 'main' into dev 2022-11-11 16:54:07 -07:00
Todd Beets
aae1d62378 test for unique_tag present in response 2022-11-11 15:26:01 -08:00
Ivan Kozlovic
da287b0aea [FIXED] Weighted subject mappings updates not applied 
Suppose an account is updated to have the following weighted mapping:
```
foo -> bar 40%
```
The server automatically adds foo -> foo at 60%. Sending messages
to "foo" will result in the expected distribution of 60% messages
going to "foo" and 40% going to bar.

However, if a successive update is pushed to the server(s):
```
foo -> bar 40%
foo -> baz 60%
```
The subject mapping should now be as described, that is, no more
mapping from "foo" to "foo" and 40% to bar and 60% to baz, however,
what was happening is that the server would always use the original
mapping.

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
 2022-11-11 15:45:32 -07:00
Todd Beets
571a375f2e add unique_tag to JetStreamConfig for varz/jsz visibility 2022-11-11 13:36:13 -08:00
Derek Collison
14ce9192f3 Merge branch 'main' into dev 2022-11-09 19:46:52 -08:00
Derek Collison
4dab6ce92c Fix test timing 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-09 19:44:22 -08:00
Ivan Kozlovic
837b36bf5f Merge branch 'main' into dev 2022-11-09 18:08:28 -07:00
Ivan Kozlovic
304744ce08 Merge pull request #3615 from nats-io/js_acc_max_streams_consumers 
[FIXED] JetStream: Account max streams/consumers not always honoured
 2022-11-09 18:02:51 -07:00
Ivan Kozlovic
1b892837cb [FIXED] JetStream: Account max streams/consumers not always honoured 
This could happen during concurrent requests where the assignments
are not yet fully processed.

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
 2022-11-09 17:29:20 -07:00
Derek Collison
e008e015b3 Make sure to enforce HA asset limits during peer processing as well as assignment. 
Signed-off-by: Derek Collison <derek@nats.io>
 2022-11-09 16:24:54 -08:00
Ivan Kozlovic
cf7c6a373c Merge branch 'main' into dev 2022-11-08 09:57:22 -07:00
Ivan Kozlovic
3ec42d5b85 Updates to PR #3611 
- Save the TLS name only if not already set
- Use the passed URLs slice instead of using s.getOpts().Routes
- Enhanced the test
- Fixed an unrelated DATA RACE report

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
 2022-11-08 09:36:08 -07:00
Ivan Kozlovic
ea955b23a1 Merge branch 'main' into dev 2022-11-07 17:48:40 -07:00
Ivan Kozlovic
2d181e1c27 [FIXED] Routing: TLS connections to discovered server may fail 
The server was not setting "server name" in the TLS configuration
for route connections, which may lead to failed (re)connect if
the certificate does not allow for the IP and the URL did not
have the hostname, which would happen with gossip protocol.

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
 2022-11-07 17:26:17 -07:00