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

feat/event-listeners #220

Merged
merged 20 commits into from
Oct 31, 2024
Merged

feat/event-listeners #220

merged 20 commits into from
Oct 31, 2024

Conversation

mohitpubnub
Copy link
Contributor

@mohitpubnub mohitpubnub commented May 19, 2024

feat: support for Channel, ChannelGroup, Subscription, ChannelMetadata, UserMetadata and SubscriptionSet entities for Subscription related operations.

Added support for Channel, ChannelGroup, Subscription and SubscriptionSet, ChannelMetadata, UserMetadata entities for Subscribe related operation.

feat: new event listener syntax support.

Added new event listeners support.

BREAKING CHANGES: Dropped support for .Net versions older than 6.

feat: HttpClient as default library for all api calls.

All apis calls will be made through HttpClient by default.

BREAKING CHANGES: Default subscribe request retry policy set to EXPONENTIAL.

feat: subscribe operation will be executed through EventEngine by default.

Default value for EnableEventEngine is set to true, Subscribe operation will be executed by event engine by default.

refactor: removed telemetry data

Removed telemetry manager.

@mohitpubnub mohitpubnub marked this pull request as ready for review June 4, 2024 06:12
mohitpubnub and others added 11 commits June 6, 2024 10:53
* Remove TelemetryManager from the sdk

* Transport layer (#225)

* * Target framework upgrade
* Initial code for HttpClient

* Integration of TransportLayer with all endpoints except File and Subscribe operations

* send and download file apis refactoring

* code cleanup: removed unnecessary usings for threading.channels

* removed support for older netstandartds

* Fix build issues for Unity and UWP projects

* retire old httpWebRequest

* CI Tests Fix (#228)

* temporarily remove one assert from TestSubscribeDecryptionOnNonEncryptedMessage

* re-enable unit tests on branch

* disable stub test

* remove channels argument exception from UrlRequestBuilder

* temporarily disable tests that were causing the CI to hang

* update key-set used by tests

* Fix broken Encryption tests

* Disable global HereNow tests (for now)

* re-enable access manager dependant tests

* Revert "re-enable access manager dependant tests"

This reverts commit dff644b.

* Test increased timeouts in ThenManageMembershipShouldReturnEventInfo

* Revert "Test increased timeouts in ThenManageMembershipShouldReturnEventInfo"

This reverts commit fc129da.

* Fix potentially faulty async calls in EncryptionTests

* Increase verbosity for CI debugging

* test increasing --logger verbosity

* add a NUnit ConsoleTraceListener

* add a simple debug writeline to test logging

* Update .pubnub.yml for .Net version support (#223)

* fix encryption tests PAM issue

* Fix more of file tests

* add missing test teardowns, replace v2 PAM access grants with v3 ones in tests that got a PAM 403

* change to PAM v3 in WhenObjectChannelMetadata

* failing test debug assert

* add temp debug method for PubnubCommon values

* Change PAM v2 to v3 in more tests, more failing test debug

* update .yml test keys ids

* add debug to failing ci test

* debug on file tests

* tests yml sdk keys change

* remove old test debugs

* Remove deprecated build targets

* Fix missing UWP includes

---------

Co-authored-by: Mohit Tejani <[email protected]>

* fix acceptance tests errors

* * fixes issue reported by acceptance tests in AppContextApi.
* Added support for Patch requests in client library

* * added missing includes for UWP (missed in last merge!)
* fix current userId memenership issue for AppContext Api

* * fix: removed references of old http client code,
added request timeout mechanism,
fixes to adhere to existing response generation mechanism by applying request state values as per response

* fix: removeMembership for current user scenario execution

* refactor: moved out enumToUrlParamName conversation utility to build utility directory, moved unnecessary log messages from internet utility method

* fixes issues found in channelGroup, audit, publish, signal operation through unit/integration tests

* files: fix issue of uploading and downloading files due to wrong header information sent through default http client component

* Unit and Integration Tests: increased delay value for Grant response wait.

* * Code cleanup: removed commented code in client library, better error handling in default http client library,
removed code for UrlProcess request from PubNubCore class,
fixed issues with generate file url

* build: update default runner group value with organization

* Add/re-add ConfigureAwait(false) in await calls to prevent Unity-side deadlocks

* Add few more missing ConfigureAwait(false)

* Added Support for Porxy.
Code Cleanup for Pubnub, PubnubCoreBase, PNConfiguration class,
Code Cleanup - removed dead codes in various files and removed old PubnubHttpClient footprints.

* fix: Serialization failure for User publish/signal message at NewtonsoftJsonNet library

* BREAKING CHANGE: Default Reconnection Policy set to `EXPONENTIAL`.
* File upload failure edge cases handling with proper exception messages.
* Refactored Check Internet status execution a bit (many more to come) to fix Unity engine impact.
* Old subscription logic cleanup: removing many state/internetstatus information store

* Unsubscribe operation refactoring: removed unncessary processings

* set default event engine enabled

* removed timer.cs, tuple.cs which was there for older .net versions

* fix: presence tests (passing locally)

* reverted eventengine enabled flag reset for Presence Tests

* update test to have different channel names for presence tests

* presence test fix for 403 permission

* attempt to fix unit/integartion test for subscribe

* fix: Patch work for internet status chekcing execution to make it compatible with single threaded environment.
Disable ee for unit/integration tests

---------

Co-authored-by: PUBNUB\jakub.grzesiowski <[email protected]>
Co-authored-by: jakub-grzesiowski <[email protected]>

---------

Co-authored-by: PUBNUB\jakub.grzesiowski <[email protected]>
Co-authored-by: jakub-grzesiowski <[email protected]>
…bnub constructor for middleware and client library
Copy link
Contributor

@jakub-grzesiowski jakub-grzesiowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@mohitpubnub
Copy link
Contributor Author

@pubnub-release-bot release as v7.0.0.0

@mohitpubnub mohitpubnub merged commit 7f4180c into master Oct 31, 2024
5 checks passed
@mohitpubnub mohitpubnub deleted the CLEN-1851 branch October 31, 2024 03:45
@pubnub-release-bot
Copy link
Contributor

🚀 Release successfully completed 🚀

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

Successfully merging this pull request may close these issues.

3 participants