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: removed private key reference and from at_lookup_impl #451

Merged
merged 5 commits into from
Dec 5, 2023

Conversation

purnimavenkatasubbu
Copy link
Member

@purnimavenkatasubbu purnimavenkatasubbu commented Nov 15, 2023

- What I did

  • Removed privateKey reference in the at_lookup_impl
  • Remove piece of code that used a private key class variable to perform auth

- How to verify it

  • Manual testing has been done with at_client/at_client_mobile and all the apps
  • existing unit/functional/e2e tests passing should count as verification

- Description for the changelog

fix: removed private key reference and from at_lookup_impl

@srieteja srieteja changed the title fix: removed private key reference fix: removed private key reference and from at_lookup_impl Dec 4, 2023
@srieteja srieteja marked this pull request as ready for review December 4, 2023 14:32
@murali-shris murali-shris merged commit b571d24 into trunk Dec 5, 2023
11 checks passed
@murali-shris murali-shris deleted the issue_434 branch December 5, 2023 13:58
@@ -573,14 +574,11 @@ class AtLookupImpl implements AtLookUp {
if (_atChops != null) {
logger.finer('calling pkam using atchops');
await pkamAuthenticate(enrollmentId: enrollmentId);
} else if (privateKey != null) {
logger.finer('calling pkam without atchops');
await authenticate(privateKey);
Copy link
Contributor

Choose a reason for hiding this comment

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

The above lines of code should not have been removed @sitaram-kalluri

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