Gary Casey comments

Results 395 comments of


                                            Gary Casey

Investigate ability to detect out-of-memory-related restart looping

I will create a script during this sprint and do some testing via my atServer to verify it

feat: <breaking_change> persistence spec refactoring to replace metadata params with metadata object from at_commons

@murali-shris What's the status of this PR? Should it be merged, or closed?

notify verb should work on authenticated connections only

A better solution here might be to overhaul the verb permission structure so instead of 'requiresAuth' we flip it to say whether or not the owner, or another atServer, or...

fix: add enrollment details to local keystore

@murali-shris Has this been superseded by #599 ? If so, please close this one.

Setting `isEncrypted:false` in notify: verb doesn't actually set it to `false`.

Hi @srieteja I've been looking at this as it came up in another context. I'll pick it up this sprint. Summary of the current behaviour for 'update' notifications - at_secondary_server...

Setting `isEncrypted:false` in notify: verb doesn't actually set it to `false`.

See also https://github.com/atsign-foundation/at_client_sdk/issues/1332

Setting `isEncrypted:false` in notify: verb doesn't actually set it to `false`.

- [at_server part](https://github.com/atsign-foundation/at_server/pull/1983) is complete, pending review and merge - [at_client part](https://github.com/atsign-foundation/at_client_sdk/issues/1332) will be completed this sprint

Setting `isEncrypted:false` in notify: verb doesn't actually set it to `false`.

- [at_server part](https://github.com/atsign-foundation/at_server/pull/1983) is complete, have requested review - [at_client part](https://github.com/atsign-foundation/at_client_sdk/issues/1332) will be merged once the at_server part has been rolled out

apkam public key for storing which apps can approve enrollment

We never agreed on the best way forward for this. Reverting to Triage, marking as a Proposal for discussion in upcoming sprint(s)

Enhancement: Enforcing Superset Access Check for Approving Apps

@sitaram-kalluri Good idea, makes sense to me