-
Notifications
You must be signed in to change notification settings - Fork 94
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
NullReferenceException in CreateSession #108
Comments
I tried to make the same call to API from Azure Diagnostic Console using powershell script, and also got "The underlying connection was closed" error. Interesting, that error for powershell script is being received on both servers -- where I get nullref and where everything works fine. |
I'm experiencing the same issue on an AWS EC2 instance. I was working on the development server for a production app that had some other changes related to Google Maps and was checking everything else when the error occurred. The TokBox code has been running for several years with no problems, so I opened the sample project and the same error occurred. I'll try it locally and see what happens. |
It looks like the solution is a single line of code to force TLS 1.2 before creating a session, and also from what I found, starting an archive. There was actually a note on this in one of the TokBox support areas about two weeks ago. |
@eccdev yes, it is. One day of pulling my hear out has been fixed by one line :) |
Are there any reasons of why this line should not be in opentok SDK? |
here is the link to support article https://support.tokbox.com/hc/en-us/articles/360000373020-Getting-error-while-creating-session-using-OpenTok-Net-SDK |
It's still somewhat incomplete. It seems like it's also needed for archiving calls, even the one to get archive info. It's funny that they don't mention going from ASP.NET 4.5.x to 4.6, and in the case of a large solution, this can be a headache. |
@skalinets Thanks for reporting the issue and sorry for the delayed response my end. As of Aug 1st, the our servers only accept TLS1.1 and above. This means that any API call that's made to OpenTok servers must be done using TLS1.1 or above. We haven't hard coded that in the SDK because we don't want to limit our users to TLS 1.1 or TLS1.2 (it should be picking up from the server environment). You should be able to add it to your |
@msach22 I think 2 changes would help someone who will face this or similar issue
|
That sounds like a great idea @skalinets! If you're up for it, please submit a PR, if not I will try to add this in the near future 😄 |
…tomized exception on WebExceptionStatus.SendFailure errors (usually from TLS errors). Fixes opentok#108.
@llauderesv, Could you please upgrade to the latest OpenTok .NET SDK - we are currently on v3.1.0? |
@msach22 If I upgrade to the latest version of OpenTok I think theres a need to upgrade a .NET Framework. Currently we're using .NET Framework version 4.0 in the production that's the reason why I use OpenTok 2.2.3 to support our current .NET framework. |
@llauderesv As far as I know, .NET 4.0 does not support TLS 1.1 or TLS 1.2 which is why you would have to upgrade to 4.5 at the very minimum. OpenTok does not support TLS 1.0 so you have use to TLS 1.1 or TLS 1.2 to communicate with our API servers. |
Hi @msach22, I already tried to work in OpenTok v3.1.0 in my Application and I encounter this error. I also forcing to use the TLS1.2 Kindly see the screenshot of my error. Thanks. |
@llauderesv Could you please confirm that you application is running .NET 4.5? I recommend running the HelloWorld sample within this repo to see if it works for you. https://github.com/opentok/Opentok-.NET-SDK/tree/master/Samples/HelloWorld |
This is a serious issue because the SDK does not work out-of-the-box. This creates incident and puts feeling of flakey SDK onto your users. I have read comment #108 (comment) which states:
I agree that the line should not be in SDK itself, but it should be in caller code, with a comment stating when to remove it. Having a SDK that work out-of-the-box is very important. |
@fidergo-stephane-gourichon That's a fair point. What if we updated our samples and the README of the SDK? This way the TLS code would not be in the code of the SDK, but would show you how to make it work in your app. |
How is it going @msach22 I've tried forcing SecurityProtocol to Tls12 as seen in the image. It still fails for 5 different combinations I tried. I can confirm I'm using OpenTok 3.1.0 i got from Nuget and my project is .NETCore which supports Tls12. Am I missing something? Appreciate your help!! |
Hi Everyone, We are experiencing the following error in creating session, hoping for your quick response to what might be causing this and possible solution, Thank you The underlying connection was closed: Could not establish trust relationship for the SSL/TLS secure channel. Note: We have a multiple instance of web apps hosted in the same server, two of them uses opentok capability to have a realtime video chat and one of them is successfully using it the reason why we are getting confused why the other app is having the issue above, whilst they are using the same key and secret and request configurations and SSL certificate. |
@nnaig08 Thanks for raising this. Could you verify the time clock on the application that's not working is up to date? |
Hi @msach22 yup, the time is in sync and up to date. We are really out of options regarding this issue, had rendered almost half day searching for possible solutions but still no avail. :( |
Hi @nnaig08 - Sorry for the delay in getting back to you, the issue you're referring to is almost certainly not an issue with the SDK or even anything in your code - it's almost certainly an issue with either the cert store on the server or VM that you're running on or with how IIS is configured. Our API endpoints uses GoDaddy as a root CA - which is typically installed on windows by default. Can you ensure that those GoDaddy's certs are installed on your machines? thanks! |
* Better handle WebExceptions that don't have a Response and send a customized exception on WebExceptionStatus.SendFailure errors (usually from TLS errors). Fixes #108. * Adding extra instructions regarding TLS to readme, adding new exception handling if TLS is incorrect * removing erroneously added whitespace * list archive by session id, better exception handling for validateSessionid * removing extra whitespace from tests * cleaning up other whitespace issues * passing inner exception to OpenTokException * Fixing comments, removing redundant fields * adding custom layout for archiving * fixing spacing issue, adding javadocs to StartArchive for layout * revving to 3.4 * adding extra Archive Test * Adding non-custom with stylesheet case * adding contract annotations to ArchiveLayout to prevent invalid serialization of the stylesheet for non-custom layouts * bringing balance to the force * Dropping core 3.0 tests * moving back to newer container * removing .net core 2.x runs * Silenced warnings in the test project. * empty Statement Redundancy Fix * Spell Fix * Spell Fix * Spell Fix | GetPartnerIdFromSessionId Spell Fix | GetPartnerIdFromSessionId method's Property * moving dotnet version * specifying test framework * Changed Javadoc comments to XML comments. * Replace <see href with <a href. * Added additional comments. * Revving to 3.5.0 * automatic release on new GitHub release * adding OUTPUT_PATH * fixing Windows CI issue * updating nexmo/github-actions branch to main * screenshare layout * more tests * Adding SetArchiveLayout, updating unit tests * moving null ignore to data structure * fixing error message * fixing screenshareType * revving to 3.6.0 * Removing travis config * Removing travis, fixing GitHub Actions definitions and adding package settings in project file (#164) * Docs corrections (#161) Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: Matt Hunt <mattlethargic@gmail.com> Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com>
As the last comment on this issue is approaching two years ago and with the fact that there have been two merged PRs to aid in fixing this, I am now going to close. If there are more outstanding issues relating to TLS versions please re-open or create a new issue. |
* Correction to docs for automatic archiving * Fixes NullReferenceException in CreateSession (#114) * Better handle WebExceptions that don't have a Response and send a customized exception on WebExceptionStatus.SendFailure errors (usually from TLS errors). Fixes #108. * Adding extra instructions regarding TLS to readme, adding new exception handling if TLS is incorrect * removing erroneously added whitespace * list archive by session id, better exception handling for validateSessionid * removing extra whitespace from tests * cleaning up other whitespace issues * passing inner exception to OpenTokException * Fixing comments, removing redundant fields * adding custom layout for archiving * fixing spacing issue, adding javadocs to StartArchive for layout * revving to 3.4 * adding extra Archive Test * Adding non-custom with stylesheet case * adding contract annotations to ArchiveLayout to prevent invalid serialization of the stylesheet for non-custom layouts * bringing balance to the force * Dropping core 3.0 tests * moving back to newer container * removing .net core 2.x runs * Silenced warnings in the test project. * empty Statement Redundancy Fix * Spell Fix * Spell Fix * Spell Fix | GetPartnerIdFromSessionId Spell Fix | GetPartnerIdFromSessionId method's Property * moving dotnet version * specifying test framework * Changed Javadoc comments to XML comments. * Replace <see href with <a href. * Added additional comments. * Revving to 3.5.0 * automatic release on new GitHub release * adding OUTPUT_PATH * fixing Windows CI issue * updating nexmo/github-actions branch to main * screenshare layout * more tests * Adding SetArchiveLayout, updating unit tests * moving null ignore to data structure * fixing error message * fixing screenshareType * revving to 3.6.0 * Removing travis config * Removing travis, fixing GitHub Actions definitions and adding package settings in project file (#164) * Docs corrections (#161) Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: Matt Hunt <mattlethargic@gmail.com> Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Implementing Dial method for initiating a SIP call (#166) * Adding Dial method, test fails * Adding dial async and tests * Adding session validation * Removing key from appsettings * Adding valid session id to tests * Docs edits for Dial API ... and other minor docs corrections Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Rev the version to 3.7.0 * Adding Force and Disable Mute (#167) * Adding Force and Disable Mute * Docs edits for the Force Mute API enhancements * Fixing PlayDTMF methods that had wrong URL Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Docs edits for v3.7.0 (#169) * Changing version number (#170) * Docs corrections Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: Paul Ardeleanu <paul.ardeleanu@vonage.com> Co-authored-by: Patrick Childers <patrick@aliihealthcare.com> Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com>
* Correction to docs for automatic archiving * Fixes NullReferenceException in CreateSession (#114) * Better handle WebExceptions that don't have a Response and send a customized exception on WebExceptionStatus.SendFailure errors (usually from TLS errors). Fixes #108. * Adding extra instructions regarding TLS to readme, adding new exception handling if TLS is incorrect * removing erroneously added whitespace * list archive by session id, better exception handling for validateSessionid * removing extra whitespace from tests * cleaning up other whitespace issues * passing inner exception to OpenTokException * Fixing comments, removing redundant fields * adding custom layout for archiving * fixing spacing issue, adding javadocs to StartArchive for layout * revving to 3.4 * adding extra Archive Test * Adding non-custom with stylesheet case * adding contract annotations to ArchiveLayout to prevent invalid serialization of the stylesheet for non-custom layouts * bringing balance to the force * Dropping core 3.0 tests * moving back to newer container * removing .net core 2.x runs * Silenced warnings in the test project. * empty Statement Redundancy Fix * Spell Fix * Spell Fix * Spell Fix | GetPartnerIdFromSessionId Spell Fix | GetPartnerIdFromSessionId method's Property * moving dotnet version * specifying test framework * Changed Javadoc comments to XML comments. * Replace <see href with <a href. * Added additional comments. * Revving to 3.5.0 * automatic release on new GitHub release * adding OUTPUT_PATH * fixing Windows CI issue * updating nexmo/github-actions branch to main * screenshare layout * more tests * Adding SetArchiveLayout, updating unit tests * moving null ignore to data structure * fixing error message * fixing screenshareType * revving to 3.6.0 * Removing travis config * Removing travis, fixing GitHub Actions definitions and adding package settings in project file (#164) * Docs corrections (#161) Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: Matt Hunt <mattlethargic@gmail.com> Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Implementing Dial method for initiating a SIP call (#166) * Adding Dial method, test fails * Adding dial async and tests * Adding session validation * Removing key from appsettings * Adding valid session id to tests * Docs edits for Dial API ... and other minor docs corrections Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Rev the version to 3.7.0 * Adding Force and Disable Mute (#167) * Adding Force and Disable Mute * Docs edits for the Force Mute API enhancements * Fixing PlayDTMF methods that had wrong URL Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Docs edits for v3.7.0 (#169) * Changing version number (#170) * Docs corrections Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: Paul Ardeleanu <paul.ardeleanu@vonage.com> Co-authored-by: Patrick Childers <patrick@aliihealthcare.com> Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com>
* Correction to docs for automatic archiving * Fixes NullReferenceException in CreateSession (#114) * Better handle WebExceptions that don't have a Response and send a customized exception on WebExceptionStatus.SendFailure errors (usually from TLS errors). Fixes #108. * Adding extra instructions regarding TLS to readme, adding new exception handling if TLS is incorrect * removing erroneously added whitespace * list archive by session id, better exception handling for validateSessionid * removing extra whitespace from tests * cleaning up other whitespace issues * passing inner exception to OpenTokException * Fixing comments, removing redundant fields * adding custom layout for archiving * fixing spacing issue, adding javadocs to StartArchive for layout * revving to 3.4 * adding extra Archive Test * Adding non-custom with stylesheet case * adding contract annotations to ArchiveLayout to prevent invalid serialization of the stylesheet for non-custom layouts * bringing balance to the force * Dropping core 3.0 tests * moving back to newer container * removing .net core 2.x runs * Silenced warnings in the test project. * empty Statement Redundancy Fix * Spell Fix * Spell Fix * Spell Fix | GetPartnerIdFromSessionId Spell Fix | GetPartnerIdFromSessionId method's Property * moving dotnet version * specifying test framework * Changed Javadoc comments to XML comments. * Replace <see href with <a href. * Added additional comments. * Revving to 3.5.0 * automatic release on new GitHub release * adding OUTPUT_PATH * fixing Windows CI issue * updating nexmo/github-actions branch to main * screenshare layout * more tests * Adding SetArchiveLayout, updating unit tests * moving null ignore to data structure * fixing error message * fixing screenshareType * revving to 3.6.0 * Removing travis config * Removing travis, fixing GitHub Actions definitions and adding package settings in project file (#164) * Docs corrections (#161) Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: Matt Hunt <mattlethargic@gmail.com> Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Implementing Dial method for initiating a SIP call (#166) * Adding Dial method, test fails * Adding dial async and tests * Adding session validation * Removing key from appsettings * Adding valid session id to tests * Docs edits for Dial API ... and other minor docs corrections Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Rev the version to 3.7.0 * Adding Force and Disable Mute (#167) * Adding Force and Disable Mute * Docs edits for the Force Mute API enhancements * Fixing PlayDTMF methods that had wrong URL Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Docs edits for v3.7.0 (#169) * Changing version number (#170) * Docs corrections * Selective adding of streams for archive and broadcast (#174) * Selective adding of streams for archive and broadcast * Minor docs edits for StreamMode additions * bumping version to 3.8.0 * Another version bump Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Devx 5778 (#179) * Moving tests * refactoring test base * CreateSession async * Start Archive Async * Fixing dial issue * Removing duplicate version tags in proj file and updating README with Async methods Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Minor docs typo correction (#181) * Adding GetArchive and DeleteArchive async methods (#182) * Adding GetArchive and DeleteArchive async methods * Adding readme info and adding methods to Archieve object * DVR and LowLatency changes added (#183) * DVR and LowLatency changes added * Update OpenTok/OpenTok.cs Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Fixing comments for PR * Adding Broadcast settings summary * Docs edits for BroadcastHlsSettings and related content. Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: Paul Ardeleanu <paul.ardeleanu@vonage.com> Co-authored-by: Patrick Childers <patrick@aliihealthcare.com> Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com>
* Correction to docs for automatic archiving * Fixes NullReferenceException in CreateSession (#114) * Better handle WebExceptions that don't have a Response and send a customized exception on WebExceptionStatus.SendFailure errors (usually from TLS errors). Fixes #108. * Adding extra instructions regarding TLS to readme, adding new exception handling if TLS is incorrect * removing erroneously added whitespace * list archive by session id, better exception handling for validateSessionid * removing extra whitespace from tests * cleaning up other whitespace issues * passing inner exception to OpenTokException * Fixing comments, removing redundant fields * adding custom layout for archiving * fixing spacing issue, adding javadocs to StartArchive for layout * revving to 3.4 * adding extra Archive Test * Adding non-custom with stylesheet case * adding contract annotations to ArchiveLayout to prevent invalid serialization of the stylesheet for non-custom layouts * bringing balance to the force * Dropping core 3.0 tests * moving back to newer container * removing .net core 2.x runs * Silenced warnings in the test project. * empty Statement Redundancy Fix * Spell Fix * Spell Fix * Spell Fix | GetPartnerIdFromSessionId Spell Fix | GetPartnerIdFromSessionId method's Property * moving dotnet version * specifying test framework * Changed Javadoc comments to XML comments. * Replace <see href with <a href. * Added additional comments. * Revving to 3.5.0 * automatic release on new GitHub release * adding OUTPUT_PATH * fixing Windows CI issue * updating nexmo/github-actions branch to main * screenshare layout * more tests * Adding SetArchiveLayout, updating unit tests * moving null ignore to data structure * fixing error message * fixing screenshareType * revving to 3.6.0 * Removing travis config * Removing travis, fixing GitHub Actions definitions and adding package settings in project file (#164) * Docs corrections (#161) Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: Matt Hunt <mattlethargic@gmail.com> Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Implementing Dial method for initiating a SIP call (#166) * Adding Dial method, test fails * Adding dial async and tests * Adding session validation * Removing key from appsettings * Adding valid session id to tests * Docs edits for Dial API ... and other minor docs corrections Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Rev the version to 3.7.0 * Adding Force and Disable Mute (#167) * Adding Force and Disable Mute * Docs edits for the Force Mute API enhancements * Fixing PlayDTMF methods that had wrong URL Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Docs edits for v3.7.0 (#169) * Changing version number (#170) * Docs corrections * Selective adding of streams for archive and broadcast (#174) * Selective adding of streams for archive and broadcast * Minor docs edits for StreamMode additions * bumping version to 3.8.0 * Another version bump Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Devx 5778 (#179) * Moving tests * refactoring test base * CreateSession async * Start Archive Async * Fixing dial issue * Removing duplicate version tags in proj file and updating README with Async methods Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Minor docs typo correction (#181) * Adding GetArchive and DeleteArchive async methods (#182) * Adding GetArchive and DeleteArchive async methods * Adding readme info and adding methods to Archieve object * DVR and LowLatency changes added (#183) * DVR and LowLatency changes added * Update OpenTok/OpenTok.cs Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Fixing comments for PR * Adding Broadcast settings summary * Docs edits for BroadcastHlsSettings and related content. Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Devx 5971 - StopArchive and ListArchive Async Methods (#184) * Adding Stop Archive and ListArchive async methods * Minor docs edit Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Adding StopBroadcast and GetBroadcast methods (#186) * Adding GetStream and ListStreams async methods (#187) * Adding SetBroadcastLayout and SetArchiveLayout async methods (#188) * Adding SetBroadcastLayout and SetArchiveLayout async methods * Minor docs edit * Fixing merge conflict issues Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Fixing some merge issues Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: Paul Ardeleanu <paul.ardeleanu@vonage.com> Co-authored-by: Patrick Childers <patrick@aliihealthcare.com> Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com>
* Correction to docs for automatic archiving * Fixes NullReferenceException in CreateSession (#114) * Better handle WebExceptions that don't have a Response and send a customized exception on WebExceptionStatus.SendFailure errors (usually from TLS errors). Fixes #108. * Adding extra instructions regarding TLS to readme, adding new exception handling if TLS is incorrect * removing erroneously added whitespace * list archive by session id, better exception handling for validateSessionid * removing extra whitespace from tests * cleaning up other whitespace issues * passing inner exception to OpenTokException * Fixing comments, removing redundant fields * adding custom layout for archiving * fixing spacing issue, adding javadocs to StartArchive for layout * revving to 3.4 * adding extra Archive Test * Adding non-custom with stylesheet case * adding contract annotations to ArchiveLayout to prevent invalid serialization of the stylesheet for non-custom layouts * bringing balance to the force * Dropping core 3.0 tests * moving back to newer container * removing .net core 2.x runs * Silenced warnings in the test project. * empty Statement Redundancy Fix * Spell Fix * Spell Fix * Spell Fix | GetPartnerIdFromSessionId Spell Fix | GetPartnerIdFromSessionId method's Property * moving dotnet version * specifying test framework * Changed Javadoc comments to XML comments. * Replace <see href with <a href. * Added additional comments. * Revving to 3.5.0 * automatic release on new GitHub release * adding OUTPUT_PATH * fixing Windows CI issue * updating nexmo/github-actions branch to main * screenshare layout * more tests * Adding SetArchiveLayout, updating unit tests * moving null ignore to data structure * fixing error message * fixing screenshareType * revving to 3.6.0 * Removing travis config * Removing travis, fixing GitHub Actions definitions and adding package settings in project file (#164) * Docs corrections (#161) Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: Matt Hunt <mattlethargic@gmail.com> Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Implementing Dial method for initiating a SIP call (#166) * Adding Dial method, test fails * Adding dial async and tests * Adding session validation * Removing key from appsettings * Adding valid session id to tests * Docs edits for Dial API ... and other minor docs corrections Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Rev the version to 3.7.0 * Adding Force and Disable Mute (#167) * Adding Force and Disable Mute * Docs edits for the Force Mute API enhancements * Fixing PlayDTMF methods that had wrong URL Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Docs edits for v3.7.0 (#169) * Changing version number (#170) * Docs corrections * Selective adding of streams for archive and broadcast (#174) * Selective adding of streams for archive and broadcast * Minor docs edits for StreamMode additions * bumping version to 3.8.0 * Another version bump Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Devx 5778 (#179) * Moving tests * refactoring test base * CreateSession async * Start Archive Async * Fixing dial issue * Removing duplicate version tags in proj file and updating README with Async methods Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Minor docs typo correction (#181) * Adding GetArchive and DeleteArchive async methods (#182) * Adding GetArchive and DeleteArchive async methods * Adding readme info and adding methods to Archieve object * DVR and LowLatency changes added (#183) * DVR and LowLatency changes added * Update OpenTok/OpenTok.cs Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Fixing comments for PR * Adding Broadcast settings summary * Docs edits for BroadcastHlsSettings and related content. Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Devx 5971 - StopArchive and ListArchive Async Methods (#184) * Adding Stop Archive and ListArchive async methods * Minor docs edit Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Adding StopBroadcast and GetBroadcast methods (#186) * Adding GetStream and ListStreams async methods (#187) * Adding SetBroadcastLayout and SetArchiveLayout async methods (#188) * Adding SetBroadcastLayout and SetArchiveLayout async methods * Minor docs edit * Fixing merge conflict issues Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Fixing some merge issues * Devx 5946 (#191) * added SetStreamListsAsync * SignalAsync methods added * Adding StartBroadcast async methods Refactoring tests * Devx 5915 - Update .NET Server SDK for 1080p Archive/Broadcast (#207) * Add all available resolutions for archiving in README.md * Add all available resolutions on XML documentation for Broadcast.cs * Add all available resolutions on Broadcasting sample app * Upgrade System.Net.Http to 4.3.2 as suggested by Whitesource * Upgrade System.Net.Http to 4.3.4 as suggested by Whitesource * Initialize publisherOptions for HelloWorld sample (#209) * DEVX-6497 - Add Experience Composer to the OpenTok .NET Server SDK (#210) * Create object and implement parsing for StartRender operation * Fix wrong allowed resolution * Implement request creation with various values, including optional ones * Fix warning for http (test value) * Kill last mutants for threshold values * Implement POST call to start a rendering * Make StartRenderResponse immutable * Implement StopRenderAsync * Implement ListRendersRequest parsing * Kill mutants for http headers * Fix type in ListRenders * Implement ListRendersAsync * Implement GetRenderAsync * Kill mutants on StartRenderRequest, make all responses as struct for value equality * Fix request formatting issue - last test round * Add missing Xml comments * Amend TODO comments * Experience Composer Render API docs edits * Make ScreenResolutionConverter private * Rename ScreenResolution into RenderResolution * Remove StatusCallbackUrl * Make StreamName optional, expect a PublisherProperties when creating a StartRenderRequest Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Devx 6491 - Update.NET Server SDK for Multiple Archives and Records (#208) * Add multiArchiveTag on Archive.cs * Fix wrong assertion in ArchiveTests.cs * Add multiBroadcastTag on Broadcast.cs * Add missing Xml documentation for new parameters * Upgrade System.Net.Http to 4.3.4 as suggested by Whitesource * Docs edits for simultaneous archive/broadcast features * Fix conflict with MultiBroadcastTag Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Rev version to 3.9.0 (#212) * DEVX-5909 - Add Audio Streamer to OpenTok .Net SDK (#211) * Implement ConnectRequest parsing * Implement StopAudioStreamer * Renaming elements for AudioConnector, fix xml documentations, flatten namespaces Rename AudioStreamer by AudioConnector Rename ConnectRequest by AudioConnectorStartRequest Rename ConnectResponse by AudioConnector * Update README with AudioConnector snippet * Fix merge conflicts * README edits * Minor docs edit --------- Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> --------- Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: Paul Ardeleanu <paul.ardeleanu@vonage.com> Co-authored-by: Patrick Childers <patrick@aliihealthcare.com> Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: Matt Hunt <mattlethargic@gmail.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com>
* Correction to docs for automatic archiving * Fixes NullReferenceException in CreateSession (#114) * Better handle WebExceptions that don't have a Response and send a customized exception on WebExceptionStatus.SendFailure errors (usually from TLS errors). Fixes #108. * Adding extra instructions regarding TLS to readme, adding new exception handling if TLS is incorrect * removing erroneously added whitespace * list archive by session id, better exception handling for validateSessionid * removing extra whitespace from tests * cleaning up other whitespace issues * passing inner exception to OpenTokException * Fixing comments, removing redundant fields * adding custom layout for archiving * fixing spacing issue, adding javadocs to StartArchive for layout * revving to 3.4 * adding extra Archive Test * Adding non-custom with stylesheet case * adding contract annotations to ArchiveLayout to prevent invalid serialization of the stylesheet for non-custom layouts * bringing balance to the force * Dropping core 3.0 tests * moving back to newer container * removing .net core 2.x runs * Silenced warnings in the test project. * empty Statement Redundancy Fix * Spell Fix * Spell Fix * Spell Fix | GetPartnerIdFromSessionId Spell Fix | GetPartnerIdFromSessionId method's Property * moving dotnet version * specifying test framework * Changed Javadoc comments to XML comments. * Replace <see href with <a href. * Added additional comments. * Revving to 3.5.0 * automatic release on new GitHub release * adding OUTPUT_PATH * fixing Windows CI issue * updating nexmo/github-actions branch to main * screenshare layout * more tests * Adding SetArchiveLayout, updating unit tests * moving null ignore to data structure * fixing error message * fixing screenshareType * revving to 3.6.0 * Removing travis config * Removing travis, fixing GitHub Actions definitions and adding package settings in project file (#164) * Docs corrections (#161) Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: Matt Hunt <mattlethargic@gmail.com> Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Implementing Dial method for initiating a SIP call (#166) * Adding Dial method, test fails * Adding dial async and tests * Adding session validation * Removing key from appsettings * Adding valid session id to tests * Docs edits for Dial API ... and other minor docs corrections Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Rev the version to 3.7.0 * Adding Force and Disable Mute (#167) * Adding Force and Disable Mute * Docs edits for the Force Mute API enhancements * Fixing PlayDTMF methods that had wrong URL Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Docs edits for v3.7.0 (#169) * Changing version number (#170) * Docs corrections * Selective adding of streams for archive and broadcast (#174) * Selective adding of streams for archive and broadcast * Minor docs edits for StreamMode additions * bumping version to 3.8.0 * Another version bump Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Devx 5778 (#179) * Moving tests * refactoring test base * CreateSession async * Start Archive Async * Fixing dial issue * Removing duplicate version tags in proj file and updating README with Async methods Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Minor docs typo correction (#181) * Adding GetArchive and DeleteArchive async methods (#182) * Adding GetArchive and DeleteArchive async methods * Adding readme info and adding methods to Archieve object * DVR and LowLatency changes added (#183) * DVR and LowLatency changes added * Update OpenTok/OpenTok.cs Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Fixing comments for PR * Adding Broadcast settings summary * Docs edits for BroadcastHlsSettings and related content. Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Devx 5971 - StopArchive and ListArchive Async Methods (#184) * Adding Stop Archive and ListArchive async methods * Minor docs edit Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Adding StopBroadcast and GetBroadcast methods (#186) * Adding GetStream and ListStreams async methods (#187) * Adding SetBroadcastLayout and SetArchiveLayout async methods (#188) * Adding SetBroadcastLayout and SetArchiveLayout async methods * Minor docs edit * Fixing merge conflict issues Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Fixing some merge issues * Devx 5946 (#191) * added SetStreamListsAsync * SignalAsync methods added * Adding StartBroadcast async methods Refactoring tests * Devx 5915 - Update .NET Server SDK for 1080p Archive/Broadcast (#207) * Add all available resolutions for archiving in README.md * Add all available resolutions on XML documentation for Broadcast.cs * Add all available resolutions on Broadcasting sample app * Upgrade System.Net.Http to 4.3.2 as suggested by Whitesource * Upgrade System.Net.Http to 4.3.4 as suggested by Whitesource * Initialize publisherOptions for HelloWorld sample (#209) * DEVX-6497 - Add Experience Composer to the OpenTok .NET Server SDK (#210) * Create object and implement parsing for StartRender operation * Fix wrong allowed resolution * Implement request creation with various values, including optional ones * Fix warning for http (test value) * Kill last mutants for threshold values * Implement POST call to start a rendering * Make StartRenderResponse immutable * Implement StopRenderAsync * Implement ListRendersRequest parsing * Kill mutants for http headers * Fix type in ListRenders * Implement ListRendersAsync * Implement GetRenderAsync * Kill mutants on StartRenderRequest, make all responses as struct for value equality * Fix request formatting issue - last test round * Add missing Xml comments * Amend TODO comments * Experience Composer Render API docs edits * Make ScreenResolutionConverter private * Rename ScreenResolution into RenderResolution * Remove StatusCallbackUrl * Make StreamName optional, expect a PublisherProperties when creating a StartRenderRequest Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Devx 6491 - Update.NET Server SDK for Multiple Archives and Records (#208) * Add multiArchiveTag on Archive.cs * Fix wrong assertion in ArchiveTests.cs * Add multiBroadcastTag on Broadcast.cs * Add missing Xml documentation for new parameters * Upgrade System.Net.Http to 4.3.4 as suggested by Whitesource * Docs edits for simultaneous archive/broadcast features * Fix conflict with MultiBroadcastTag Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Rev version to 3.9.0 (#212) * DEVX-5909 - Add Audio Streamer to OpenTok .Net SDK (#211) * Implement ConnectRequest parsing * Implement StopAudioStreamer * Renaming elements for AudioConnector, fix xml documentations, flatten namespaces Rename AudioStreamer by AudioConnector Rename ConnectRequest by AudioConnectorStartRequest Rename ConnectResponse by AudioConnector * Update README with AudioConnector snippet * Fix merge conflicts * README edits * Minor docs edit --------- Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * [DEVX-6811] Expose E2EE parameter on Session creation (#218) * Add optional parameter to enable end-to-end encryption when creating a session * Minor docs edits --------- Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * fix: audio connector (#224) * Add test for serialization * Change expectation in other test --------- Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: Paul Ardeleanu <paul.ardeleanu@vonage.com> Co-authored-by: Patrick Childers <patrick@aliihealthcare.com> Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: Matt Hunt <mattlethargic@gmail.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com>
* Correction to docs for automatic archiving * Fixes NullReferenceException in CreateSession (#114) * Better handle WebExceptions that don't have a Response and send a customized exception on WebExceptionStatus.SendFailure errors (usually from TLS errors). Fixes #108. * Adding extra instructions regarding TLS to readme, adding new exception handling if TLS is incorrect * removing erroneously added whitespace * list archive by session id, better exception handling for validateSessionid * removing extra whitespace from tests * cleaning up other whitespace issues * passing inner exception to OpenTokException * Fixing comments, removing redundant fields * adding custom layout for archiving * fixing spacing issue, adding javadocs to StartArchive for layout * revving to 3.4 * adding extra Archive Test * Adding non-custom with stylesheet case * adding contract annotations to ArchiveLayout to prevent invalid serialization of the stylesheet for non-custom layouts * bringing balance to the force * Dropping core 3.0 tests * moving back to newer container * removing .net core 2.x runs * Silenced warnings in the test project. * empty Statement Redundancy Fix * Spell Fix * Spell Fix * Spell Fix | GetPartnerIdFromSessionId Spell Fix | GetPartnerIdFromSessionId method's Property * moving dotnet version * specifying test framework * Changed Javadoc comments to XML comments. * Replace <see href with <a href. * Added additional comments. * Revving to 3.5.0 * automatic release on new GitHub release * adding OUTPUT_PATH * fixing Windows CI issue * updating nexmo/github-actions branch to main * screenshare layout * more tests * Adding SetArchiveLayout, updating unit tests * moving null ignore to data structure * fixing error message * fixing screenshareType * revving to 3.6.0 * Removing travis config * Removing travis, fixing GitHub Actions definitions and adding package settings in project file (#164) * Docs corrections (#161) Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: Matt Hunt <mattlethargic@gmail.com> Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Implementing Dial method for initiating a SIP call (#166) * Adding Dial method, test fails * Adding dial async and tests * Adding session validation * Removing key from appsettings * Adding valid session id to tests * Docs edits for Dial API ... and other minor docs corrections Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Rev the version to 3.7.0 * Adding Force and Disable Mute (#167) * Adding Force and Disable Mute * Docs edits for the Force Mute API enhancements * Fixing PlayDTMF methods that had wrong URL Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Docs edits for v3.7.0 (#169) * Changing version number (#170) * Docs corrections * Selective adding of streams for archive and broadcast (#174) * Selective adding of streams for archive and broadcast * Minor docs edits for StreamMode additions * bumping version to 3.8.0 * Another version bump Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Devx 5778 (#179) * Moving tests * refactoring test base * CreateSession async * Start Archive Async * Fixing dial issue * Removing duplicate version tags in proj file and updating README with Async methods Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Minor docs typo correction (#181) * Adding GetArchive and DeleteArchive async methods (#182) * Adding GetArchive and DeleteArchive async methods * Adding readme info and adding methods to Archieve object * DVR and LowLatency changes added (#183) * DVR and LowLatency changes added * Update OpenTok/OpenTok.cs Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Fixing comments for PR * Adding Broadcast settings summary * Docs edits for BroadcastHlsSettings and related content. Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Devx 5971 - StopArchive and ListArchive Async Methods (#184) * Adding Stop Archive and ListArchive async methods * Minor docs edit Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Adding StopBroadcast and GetBroadcast methods (#186) * Adding GetStream and ListStreams async methods (#187) * Adding SetBroadcastLayout and SetArchiveLayout async methods (#188) * Adding SetBroadcastLayout and SetArchiveLayout async methods * Minor docs edit * Fixing merge conflict issues Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com> * Fixing some merge issues * Devx 5946 (#191) * added SetStreamListsAsync * SignalAsync methods added * Adding StartBroadcast async methods Refactoring tests * Devx 5915 - Update .NET Server SDK for 1080p Archive/Broadcast (#207) * Add all available resolutions for archiving in README.md * Add all available resolutions on XML documentation for Broadcast.cs * Add all available resolutions on Broadcasting sample app * Upgrade System.Net.Http to 4.3.2 as suggested by Whitesource * Upgrade System.Net.Http to 4.3.4 as suggested by Whitesource * Initialize publisherOptions for HelloWorld sample (#209) * DEVX-6497 - Add Experience Composer to the OpenTok .NET Server SDK (#210) * Create object and implement parsing for StartRender operation * Fix wrong allowed resolution * Implement request creation with various values, including optional ones * Fix warning for http (test value) * Kill last mutants for threshold values * Implement POST call to start a rendering * Make StartRenderResponse immutable * Implement StopRenderAsync * Implement ListRendersRequest parsing * Kill mutants for http headers * Fix type in ListRenders * Implement ListRendersAsync * Implement GetRenderAsync * Kill mutants on StartRenderRequest, make all responses as struct for value equality * Fix request formatting issue - last test round * Add missing Xml comments * Amend TODO comments * Experience Composer Render API docs edits * Make ScreenResolutionConverter private * Rename ScreenResolution into RenderResolution * Remove StatusCallbackUrl * Make StreamName optional, expect a PublisherProperties when creating a StartRenderRequest Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Devx 6491 - Update.NET Server SDK for Multiple Archives and Records (#208) * Add multiArchiveTag on Archive.cs * Fix wrong assertion in ArchiveTests.cs * Add multiBroadcastTag on Broadcast.cs * Add missing Xml documentation for new parameters * Upgrade System.Net.Http to 4.3.4 as suggested by Whitesource * Docs edits for simultaneous archive/broadcast features * Fix conflict with MultiBroadcastTag Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * Rev version to 3.9.0 (#212) * DEVX-5909 - Add Audio Streamer to OpenTok .Net SDK (#211) * Implement ConnectRequest parsing * Implement StopAudioStreamer * Renaming elements for AudioConnector, fix xml documentations, flatten namespaces Rename AudioStreamer by AudioConnector Rename ConnectRequest by AudioConnectorStartRequest Rename ConnectResponse by AudioConnector * Update README with AudioConnector snippet * Fix merge conflicts * README edits * Minor docs edit --------- Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * [DEVX-6811] Expose E2EE parameter on Session creation (#218) * Add optional parameter to enable end-to-end encryption when creating a session * Minor docs edits --------- Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> * fix: audio connector (#224) * Add test for serialization * Change expectation in other test * Implement custom user-agent value (#219) * Allow broadcast to compose without audio or video. (#225) * Implement AutoArchive parameters on CreateSession (#227) --------- Co-authored-by: Jeff Swartz <jeff.swartz@vonage.com> Co-authored-by: Paul Ardeleanu <paul.ardeleanu@vonage.com> Co-authored-by: Patrick Childers <patrick@aliihealthcare.com> Co-authored-by: Stephen Lorello <42971704+slorello89@users.noreply.github.com> Co-authored-by: slorello89 <slorello89@gmail.com> Co-authored-by: Dirk Lemstra <dirk@lemstra.org> Co-authored-by: onpoc <gupta.pradeep506@gmail.com> Co-authored-by: Matt Hunt <mattlethargic@gmail.com> Co-authored-by: matt-lethargic <matt-lethargic@gmail.com>
Hi,
I have very weird issue. Following code works fine when running locally, but throws nullref when deployed to Azure web app.
Code:
Exception:
I enabled Debug mode and redirected
Console.Out
to aStringWriter
. Here is what I got when code is running in Azure:And this one is produced when the same code is running on localhost:
I thought that probably jwt token is being gerenated incorrectly in Azure and hence the error, but then I tried to run that request with the same headers, including jwt token, from localhost and it works fine.
I also tried to create .NET Web API app from scratch, deployed it to azure and it works fine as well. I deployed my impacted app to different app services and issue is being reproduced everywhere.
Even if this one is not an issue of opentok, getting nullrefs from inside 3rd party library is rather confusing. You might want to wrap it to something meaningful :)
The text was updated successfully, but these errors were encountered: