Websocket without auth will hang forever

[wenerme]

Defect

Make sure that these boxes are checked before submitting your issue -- thank you!

• Included nats.go version
• Included a [Minimal, Complete, and Verifiable example] (https://stackoverflow.com/help/mcve)

Versions of nats.go and the nats-server if one was involved:

• nats.go 1.3.0
• nats:2.7.3-alpine

OS/Container environment:

macos

Steps or code to reproduce the issue:

k -n nats port-forward svc/nats 8443:443

# hang
nats rtt -s ws://127.0.0.1:8443
# OK
nats rtt -s ws://127.0.0.1:8443 --creds /Users/wener/data/dl/nsc/nkeys/creds/KO/JS1/js-user.creds

server log

[7] 2022/03/14 11:07:36.458612 [ERR] 127.0.0.1:59596 - wid:99 - "v1.13.0:go:NATS CLI Version 0.0.30" - authentication error

Expected result:

report not authed

Actual result:

hang

[kozlovic]

@wenerme I will try to reproduce, but it would help a lot if you could show the configuration of the NATS Server and how you start it. Also, you mention using the Go client v1.3.0, but the "nats" tool uses go.mod and may have a different version. So would be good too to know "nats" specific version or git commit. Thanks!

[wenerme]

@kozlovic tried running in code using 1.3.0 and latest version, debug found the hang at processConnectInit/sendConnect/readProto .

nats --version
0.0.30

Server is started by helm

nats:
  nats:
    jetstream:
      enabled: true
      domain: xxx
      memStorage:
        enabled: false
      fileStorage:
        # bases on usecase
        enabled: true
        storageClassName: openebs-zfspv
        size: 10Gi
  exporter:
    serviceMonitor:
      enabled: true
  mqtt:
    enabled: true
  websocket:
    enabled: true
    noTLS: true
    allowedOrigins:
    - "https://xxx"
  appProtocol:
    enabled: true

  cluster:
    enabled: false

  auth:
    enabled: true
    resolver:
      type: full
      allowDelete: true
      store:
        dir: "/accounts/jwt"
        size: 1Gi
      operator: XXX
      systemAccount: XXX
      resolverPreload:
        XXX: XXX

dependencies:
- name: nats
  version: 0.14.1
  repository: https://nats-io.github.io/k8s/helm/charts

[kozlovic]

@wenerme Thanks! Yes, I was able to reproduce and will work on a fix. Stay tuned.