noports icon indicating copy to clipboard operation
noports copied to clipboard

at_activate in C

Open XavierChanth opened this issue 1 year ago • 8 comments

Dependent on atsign-foundation/at_c#176

### Tasks
- [ ] https://github.com/atsign-foundation/at_c/issues/176
- [ ] atsign-foundation/at_c#74 

XavierChanth avatar Jun 05 '24 20:06 XavierChanth

Where would this project live @XavierChanth ?

Would it be in examples/ ?

JeremyTubongbanua avatar Jun 06 '24 08:06 JeremyTubongbanua

I'm guessing here, as @XavierChanth hasn't explained why...

at_activate is a crucial component of the customer journey, and if their device needs C sshnpd then it likely also needs a smaller and more portable at_activate.

So not an example really, but also not part of the SDK. Maybe we need a utils directory, or maybe we need to put it in the noports repo alongside the daemon.

cpswan avatar Jun 06 '24 08:06 cpswan

One thing we could also consider is making an entirely new at_tools repo made in C, like at_tools_c that would house generic Atsign tools like repl, at_talk, atcat & attee (https://github.com/atsign-foundation/at_demos/issues/309), and also our at_activate.

repl and at_talk currently live in our examples/ directory.

JeremyTubongbanua avatar Jun 06 '24 12:06 JeremyTubongbanua

The primary object for creating at_activate is to make it easier to do APKAM / atSign onboarding with the C daemon. I am hesitant to put it in examples, but maybe a bin/ directory at the root of this repo. repl & at_talk can be moved into this new directory.

XavierChanth avatar Jun 06 '24 13:06 XavierChanth

This is now something that needs to float to the top of the ToDo's.. It will enable a number of customers to automate sshnpd and its keys on IoT devices without logging into the hardware.

MVP at_activate with APKAM creation of atKeys

cconstab avatar Aug 30 '24 05:08 cconstab

I will take this up this sprint @cconstab

srieteja avatar Sep 02 '24 13:09 srieteja

APKAM functionality is the target.. TY

cconstab avatar Sep 02 '24 16:09 cconstab

We would like to move this ticket to NoPorts. We would like to have at_activate be within the NoPorts repo to make things simple for our CI when we package NoPorts zips and binaries

Would it be possible to change this ticket's repo? I don't know how to do that

JeremyTubongbanua avatar Sep 06 '24 19:09 JeremyTubongbanua

Carrying forward to PR97 to implement key/keypair generation and exchange these keys with the at_server.

srieteja avatar Sep 30 '24 13:09 srieteja

@JeremyTubongbanua, @XavierChanth and I have completed the implementation of dependencies required to implement at_activate. But there is one bug which was interrupting my progress (MbedTLS read returns a connection timed out err), @JeremyTubongbanua is going to take a look at this bug today as I'm not very familiar with MbedTLS and it's eating up a lot of my time.

In the meantime, I have started working on at_activate itself. Current ETA to complete this would be Tuesday (at the latest).

srieteja avatar Oct 03 '24 14:10 srieteja

Blocked by https://github.com/atsign-foundation/at_c/issues/426

srieteja avatar Oct 14 '24 23:10 srieteja

https://github.com/atsign-foundation/at_c/issues/426 has been resolved. See ticket for explanation.

srieteja avatar Oct 15 '24 11:10 srieteja

Hows this ticket progressing.. looks like it is close but hard to tell .. Itching to test :-)

cconstab avatar Oct 21 '24 22:10 cconstab

All the implementation is complete @cconstab. Trying to resolve all the obvious bugs. You should have a version to test in a day or two.

srieteja avatar Oct 22 '24 00:10 srieteja

@purnimavenkatasubbu found a bug in an atserver that was activated with C atactivate. The server throws a publickey not found error when updating data. Looking into this.

Pending items:

  1. above mentioned bug
  2. unit tests (if there are any unit testable elements)

srieteja avatar Nov 11 '24 15:11 srieteja

The at_activate PR has been merged. The implementation to create enrollments with OTP is being tracked through https://github.com/atsign-foundation/at_c/issues/451.

srieteja avatar Dec 09 '24 15:12 srieteja

Closing this ticket as the implementation to create new enrollments using OTPs has been merged to trunk through: https://github.com/atsign-foundation/at_c/issues/451

srieteja avatar Dec 19 '24 22:12 srieteja