Umang Mundhra
Umang Mundhra
Sure @MadhushaPrasad assigning the issue to you. Please discuss how the user's will be using these errors (just the raw approach ) before submitting the PR.
> > Wrap and WithStack > > can you give me some more details about this Wrap and WithStack? @vikash @MadhushaPrasad you can refer here : https://github.com/pkg/errors?tab=readme-ov-file#adding-context-to-an-error To see what...
@MadhushaPrasad Hey! Did we came to any raw implementation on how the users will be using our errors package?
@rokerzfirst101 Can you please also update the Test function `TestSQL_getDBConnectionString`and add the test case for newly added code.
Also, it should be a better idea to log the `message` in `DEBUG` mode in case of Pub/Sub DB's.
@SUKANT007 just wanted to re-check you want to pick up this issue and work on it.... ?
Sure @SUKANT007 I agree with your thoughts of implementing the webSocket Upgrader and provide connection to user. Regarding `option to get WebSocket connection from context` ,it is not a bad...
Closing this issue as PR #1426 fixes some bits of it.
@reski-rukmantiyo Seems like you have some failing test and linters. Please fix them
@reski-rukmantiyo Hey! Hope you are still involved in the PR. Please let us know if you want any help regarding the issue or the PR. thankyou