Skip to content

Commit

Permalink
fix: include history request when re-attempting subscribe
Browse files Browse the repository at this point in the history
  • Loading branch information
palkan committed Aug 11, 2023
1 parent 0cc646d commit 4c60aec
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
2 changes: 2 additions & 0 deletions packages/core/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@

## master

- Fix re-subscription attempt to include history request.

## 0.7.4

- Add PONGs support to the extended protocol and allow passing protocol options via `protocolOptions`.
Expand Down
5 changes: 1 addition & 4 deletions packages/core/action_cable/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -86,10 +86,7 @@ export class ActionCableProtocol {
identifier
)

this.cable.send({
command: 'subscribe',
identifier
})
this.cable.send(this.buildSubscribeRequest(identifier))

this.maybeExpireSubscribe(id, identifier, retryInterval)
}, retryInterval)
Expand Down

0 comments on commit 4c60aec

Please sign in to comment.