atsign-foundation / at_client_sdk

The Dart implementation of atSDK used for implementing Atsign's technology into other software
https://pub.dev/publishers/atsign.org/packages
BSD 3-Clause "New" or "Revised" License
1.47k stars 32 forks source link

feat: APKAM Enrollment changes #1257

Closed sitaram-kalluri closed 3 months ago

sitaram-kalluri commented 4 months ago

- What I did

- How to verify it

gkc commented 4 months ago

@sitaram-kalluri will review today thank you!

gkc commented 3 months ago

@sitaram-kalluri This PR is not ready for review. Please ping me once you have addressed @murali-shris's requests and the functional tests are passing

sitaram-kalluri commented 3 months ago

@sitaram-kalluri This PR is not ready for review. Please ping me once you have addressed @murali-shris's requests and the functional tests are passing

@gkc: The review comments are addressed and updated with the latest trunk branch code.

Pending work: Once enrollment is approved, securely store the keys in the keychain manager being worked by @sachins-geekyants. Therefore, the PR is still marked as draft..

gkc commented 3 months ago

@sitaram-kalluri Have you tested with other things that use the at_client package e.g. at_talk, the various demos in at_demos, noports, etc?

gkc commented 3 months ago

@sitaram-kalluri This PR is not ready for review. Please ping me once you have addressed @murali-shris's requests and the functional tests are passing

@gkc: The review comments are addressed and updated with the latest trunk branch code.

Pending work: Once enrollment is approved, securely store the keys in the keychain manager being worked by @sachins-geekyants. Therefore, the PR is still marked as draft..

@sitaram-kalluri Since the PR is no longer marked as 'draft', can I presume that the Pending work you mention has been completed?

sitaram-kalluri commented 3 months ago

@gkc : Completed testing with at_talk and no-ports and works fine. Attaching the log

NoPorts Log : no-ports.log at_talk Log : at_talk_log

gkc commented 3 months ago

@sitaram-kalluri Functional tests failing on this test

❌ loading test/enrollment_test.dart (failed)
  Failed to load "test/enrollment_test.dart":
  test/enrollment_test.dart:309:29: Error: Method not found: 'FirstEnrollmentRequest'.
      var enrollmentRequest = FirstEnrollmentRequest(
                              ^^^^^^^^^^^^^^^^^^^^^^
sitaram-kalluri commented 3 months ago

@sitaram-kalluri Functional tests failing on this test

❌ loading test/enrollment_test.dart (failed)
  Failed to load "test/enrollment_test.dart":
  test/enrollment_test.dart:309:29: Error: Method not found: 'FirstEnrollmentRequest'.
      var enrollmentRequest = FirstEnrollmentRequest(
                              ^^^^^^^^^^^^^^^^^^^^^^

@gkc : The root cause of the failure is very clear. The "FirstEnrollmentRequest" class is not exported in the at_auth package. We did not export the class because apps will not use the FirstEnrollmentRequest. Apps will invoke the onboard method, which will internally submit the FirstEnrollmentRequest.

Something that is strange is how did tests pass till now and why the issue occur now? Will debug on this and keep it updated.

sitaram-kalluri commented 3 months ago

@sitaram-kalluri Functional tests failing on this test

❌ loading test/enrollment_test.dart (failed)
  Failed to load "test/enrollment_test.dart":
  test/enrollment_test.dart:309:29: Error: Method not found: 'FirstEnrollmentRequest'.
      var enrollmentRequest = FirstEnrollmentRequest(
                              ^^^^^^^^^^^^^^^^^^^^^^

@gkc : The root cause of the failure is very clear. The "FirstEnrollmentRequest" class is not exported in the at_auth package. We did not export the class because apps will not use the FirstEnrollmentRequest. Apps will invoke the onboard method, which will internally submit the FirstEnrollmentRequest.

Something that is strange is how did tests pass till now and why the issue occur now? Will debug on this and keep it updated.

@gkc : The issue is fixed. The issue occurs because a particular test is modified to use "FirstEnrollmentRequest". Reverted the change and modified the test as it was before.

gkc commented 3 months ago

@sitaram-kalluri Functional tests failing on this test


❌ loading test/enrollment_test.dart (failed)

  Failed to load "test/enrollment_test.dart":

  test/enrollment_test.dart:309:29: Error: Method not found: 'FirstEnrollmentRequest'.

      var enrollmentRequest = FirstEnrollmentRequest(

                              ^^^^^^^^^^^^^^^^^^^^^^

@gkc : The root cause of the failure is very clear. The "FirstEnrollmentRequest" class is not exported in the at_auth package. We did not export the class because apps will not use the FirstEnrollmentRequest. Apps will invoke the onboard method, which will internally submit the FirstEnrollmentRequest.

Something that is strange is how did tests pass till now and why the issue occur now? Will debug on this and keep it updated.

@gkc : The issue is fixed. The issue occurs because a particular test is modified to use "FirstEnrollmentRequest". Reverted the change and modified the test as it was before.

👍 thanks @sitaram-kalluri