Results 65 comments of Marco Primi

@derekcollison updated to leave `DirectGet` on, be more tolerant of stale reads, and retry with `checkFor`. (2 commits for easier review, squashable on merge)

Updated indentation in 2nd commit Added 3rd commit to switch to `require_` where appropriate

Just rebased, the Jenkins error comes from [Run non JetStream/MQTT tests from the server package](https://app.travis-ci.com/github/nats-io/nats-server/jobs/613351368)::`TestProcessSignalMultipleProcessesGlob`, the rest is green.

@derekcollison this got stale, I'll rebase. The failing RAFT i mentioned is a different (draft) PR: https://github.com/nats-io/nats-server/pull/4626

Freshly rebased on top of latest `main`.

> My URL was https://FQDN/api/greader.php, without the rss in it Ah indeed. `/rss/` is specific to my own setup (hosting multiple applications on the same server). Updated the original comment....

Parity with other clients is also why I'd (soft) suggest adding this. Tutorials, docs, tools & co may provide users with a connect string and it would be nice if...

I'd suggest the following logic (very close to what you did, but not exactly the same) New state variable: `leaderLastAETimestamp` Every time you get an AE from the node you...

Thank you for the comment. Yes I agree this would make it a little better to use. At the moment I don't have time to actively maintain and improve the...

@mluisbrown I think you are correct, according to [this](https://developer.mozilla.org/en-US/docs/Web/API/Crypto/subtle#browser_compatibility), HTTPS is required for `crypto.subtle`, and a more clear error should be presented otherwise. Keep in mind that there's always a...