Skip to content
New issue

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

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

Already on GitHub? # to your account

Handled failure in getApplicationName() #2571

Merged
merged 7 commits into from
Jan 2, 2025

Conversation

muskan124947
Copy link
Contributor

No description provided.

@muskan124947 muskan124947 self-assigned this Dec 30, 2024
machavan
machavan previously approved these changes Dec 30, 2024
machavan
machavan previously approved these changes Dec 31, 2024
machavan
machavan previously approved these changes Dec 31, 2024
machavan
machavan previously approved these changes Jan 2, 2025
@muskan124947 muskan124947 merged commit c7096c7 into main Jan 2, 2025
3 checks passed
Ananya2 pushed a commit that referenced this pull request Jan 10, 2025
* Handled failure in getApplicationName()

* Added test to validate appName using select app_name() query

* Updated connect property

* Updated connect property to add appname

* Fixed checking of APPLICATION_NAME in connect properties

* Added test case

---------

Co-authored-by: machavan <machavan@microsoft.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: Closed/Merged PRs
Development

Successfully merging this pull request may close these issues.

3 participants