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

fix: message parsing by sdk #1051

Merged
merged 1 commit into from
Jan 23, 2024
Merged

fix: message parsing by sdk #1051

merged 1 commit into from
Jan 23, 2024

Conversation

Aman035
Copy link
Member

@Aman035 Aman035 commented Jan 23, 2024

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Copy link

In the file packages/restapi/src/lib/chat/helpers/crypto.ts, I found the following issues:

  1. On line 55, there is an unused import for the Signer module.
  2. On line 56, there is a missing comma after the import statement for decryptWithWalletRPCMethod.
  3. On line 130, there is a missing closing parenthesis for the decryptFeeds function declaration.
  4. On line 133, there is a missing opening brace for the loop inside the decryptFeeds function.
  5. On line 155, there is a missing closing parenthesis for the signMessageWithPGPCore function declaration.
  6. On line 157, there is a missing opening brace for the signMessageWithPGPCore function.
  7. On line 168, there is a missing closing parenthesis for the decryptAndVerifyMessage function call.
  8. On line 169, there is a missing opening brace for the if statement.
  9. On line 193, there is a missing closing brace for the loop inside the decryptFeeds function.
  10. On line 214, there is a missing closing parenthesis for the getEncryptedRequest function declaration.
  11. On line 216, there is a missing opening brace for the getEncryptedRequest function.
  12. On line 230, there is a missing closing parenthesis for the verifySignature function call inside the else block.
  13. On line 238, there is a missing closing brace for the if statement.
  14. On line 245, there is a missing closing brace for the try block.
  15. On line 255, there is a missing closing parenthesis for the aesDecrypt function call inside the try block.
  16. On line 262, there is a missing opening brace for the try block.
  17. On line 275, there is a missing closing brace for the loop inside the try block.

Please review and make the necessary corrections.

@Aman035 Aman035 merged commit 0aa0acd into main Jan 23, 2024
1 check passed
@Aman035 Aman035 deleted the fix-message-parsing branch January 23, 2024 07:32
mishramonalisha76 added a commit that referenced this pull request Feb 13, 2024
* feat(video-v2): add highlevel video class

* feat(video): add video stream

* fix(sendnotification): modify rules.access.data to be an object & code cleanup

* fix(video): remove signer from input, throw err if signer, decrypted pgp key  not found

* feat(video): add sendNotification calls in connect, disconnect methods

* fix(videonotificationrules): typo in VideoNotificationRules interface

* add: useStream.ts video-v2-example-app

* add: videoV2.tsx in video-v2-example-app

* feat(video stream): handle connect, retry internally from the SDK

* feat(video connect): remove the connect method from the videov2 SDK

* fix(videov2): create push signer instance to get chain id in initialize()

* fix(videov2-example): refactor react example component & remove useStream hook

* fix(video stream): add backwards compatibilty to rules object for older SDK versions

* fix(video stream): fix bug in rules object creation

* fix(video stream): call connect() for ApproveVideo event

* feat(example-video-1.5): video v2  example app  implementation

* feat(sdk-frontend): few ui changes

* fix(video-v2-example-app): initialisevdeo call after a call ends

* feat(video stream): add request, deny evennt handlers

* feat(sdk-frontend and sdk-frontend-react): added video-v2 example implementation

* fix: few fixes

* refactor: refator and remove console logs

* fix: fix import error

* feat(video): update param names for video.initialize()

* fix: param name changes

* chore: added few comments

* chore: ui toast added

* fix(video example): remove pushprotocol sdk direct dir reference for initVideoCallData

* fix(video example): import React in examples & update SDK version in sdk-frontend

* feat(video example): replace video events from CONSTANTS obj

* Edited video example to make it more versatile

* Update index.js

* fix: fixed message parsing (#1051)

* Minor fixes and improvements in video v2 (#1062)

* feat(video v2): add VIDEO constants and types

* feat(video v2): modify approve, deny to use internally stored peerInfo

* fix(video constants): rename DEFAULT_DATA to INITIAL_DATA

* 1050 update video example implementation in sdk-frontend (#1070)

* fix: use latest imports in example app

* feat: use  latest rainbowkit library

* chore: use latest restapi package

* chore: remove unnecessary libs

* fix: fixed signer bug (#1073)

* fix: send intent in history and latest. (#1069)

* fix: send intent in history and latest.

* fix: get chat info API sdk changes

* fix: stream helper functions

* fix: caching implementation on user profile API (#1053)

* fix: caching implementation on user profile API

* fix: cache code review comments

* fix: fix public key in read mode (#1074)

* fix: added notification widget layout

* fix: fixed naming convention

* fix: added layout for hyperlink

* Pub key cache to speed up chat messages verification (#1081)

* fix: minor fixes

* fix: cache implementation for public keys

* fix: fix encV1 key decryption (#1088)

* fix: add benchamark tests, fix reference msgs' (#1089)

* fix: added resuables

* bolierplate examples

* bolierplate examples

* bolierplate examples

* fix: added connect button

* fix: fixed lint

* fix: v2 user creation / upgrades / downgrades (#1099)

* fix: fix v2 user

* test: remove only

* Add filter for subscription (#1102)

* fix: added parsing for settings

* fix: minor fixes

* fix: added manage notifiction

* fix: added channel setting parse logic (#1107)

* fix: fixed css

* added user downgrade example

* fix: fixed lint

* fix: fixed connect modal

* fix: added readmode

* fix: fixed readMode

---------

Co-authored-by: Madhur Gupta <madhur@push.org>
Co-authored-by: Siddesh Sankhya <79219618+Siddesh7@users.noreply.github.com>
Co-authored-by: Siddesh <siddeshsankhya@gmail.com>
Co-authored-by: harshrajat <harsh@epns.io>
Co-authored-by: Harsh | Push <harsh@push.org>
Co-authored-by: Aman Gupta <guptaaman200115@gmail.com>
Co-authored-by: Mohammed S <shoaib@push.org>
Co-authored-by: Ashis Kumar Pradhan <38760485+akp111@users.noreply.github.com>
Co-authored-by: Mohammed S <shoaibmohammed92@gmail.com>
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.

🐛 [BUG] - Message decryption fails for invalid json values in messageObj
2 participants