tsolucio / coreBOSwsLibrary

coreBOS webservice Library
9 stars 15 forks source link

NEW: Removing sessionName from request body/params #34

Closed ochu-elhadji closed 2 years ago

ochu-elhadji commented 2 years ago
joebordes commented 2 years ago

I was expecting to see in this PR some code that adds the sessionName to the header. Isn't that missing?

ochu-elhadji commented 2 years ago

I was expecting to see in this PR some code that adds the sessionName to the header. Isn't that missing?

I just want the client app to set, but adding this in the package itself makes sense. I will add it.

joebordes commented 2 years ago

great!

should we update the other two files?

ochu-elhadji commented 2 years ago

great!

should we update the other two files?

  • npm/wsclientm.js
  • wsclientp.js

If the coreBOS is ready for this, we can update.