Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not able to add cookie as parameter #15

Open
samiramin opened this issue Aug 31, 2016 · 3 comments
Open

Not able to add cookie as parameter #15

samiramin opened this issue Aug 31, 2016 · 3 comments

Comments

@samiramin
Copy link

samiramin commented Aug 31, 2016

I am trying to pass cookie as parameter with new setCookie method.
I have added method in IWebSocketConnection interface of Websockets project, and also added method in Websockets.iOS and Websockets.Droid project.

I am using PCL project for iOS and android. From my main project I am trying to call setCookie method with cookie parameter it gives error.

Error CS1061: Type Websockets.IWebSocketConnection' does not contain a definition forsetCookie' and no extension method setCookie' of typeWebsockets.IWebSocketConnection' could be found. Are you missing an assembly reference? (CS1061)

I am able to use connection.setCookie method but when I try to compile it it shows the above error.

Without setting cookie websocket is able to connect with test url i.e ws://echo.websocket.org/

I am only able to access and use the method which are already there, I am not able to add my method.
Why I am not able to access new methods?

@samiramin samiramin changed the title Not able to add parameter to add cookie Not able to add cookie as parameter Aug 31, 2016
@NVentimiglia
Copy link
Owner

https://github.com/NVentimiglia/Websockets.PCL/blob/master/Websockets/IWebSocketConnection.cs

There is no setCookie method. if you define it, I will accept a pull request

@samiramin
Copy link
Author

samiramin commented Sep 1, 2016

@NVentimiglia, I know there is no setCookie method, I have added new method and using it in my PCL project.
But to generate a pull request It should work.
My issue is I am able to use setCookie method using intellisense and when I build the app it gives an error.

@NVentimiglia
Copy link
Owner

Have this source somewhere for review ? I will accept pulls.

@NVentimiglia NVentimiglia reopened this Sep 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants