search

boucadair / CATS-framework

Other
1 stars 4 forks source link

Update draft-ietf-cats-framework.md #91

Closed muzixing closed 2 weeks ago

muzixing commented 1 month ago

update to address Zongpeng's comments

muzixing commented 3 weeks ago

This update is agreed by Cheng and Zongpeng, other authors have not replied yet. Med, if you are ok with the update, please merge it.

muzixing commented 3 weeks ago

For the rest comments，my proposal is to go with two ways:

  1. keep it now, and send an email to the list to discuss the text of definition. change the definition in two drafts next time.
  2. provide new text, update it and ask for comment. Update it again when we get conclusion on the definition.

It will be better to update this revision ASAP, so that we can work on another revision to address rest issues.

boucadair commented 3 weeks ago

I suggest we don't make proposed changes for the pending points (that is, revert to the OLD text). For each pending point, create a separate PR. That would ease merging and converging. Thanks.

muzixing commented 3 weeks ago

I suggest we don't make proposed changes for the pending points (that is, revert to the OLD text). For each pending point, create a separate PR. That would ease merging and converging. Thanks.

I fully agree. So let's do in this way.

BTW, I merged two PRs already, cause the conflicts on this. You may revert them and merge this firstly, or you can edit it manually. Sorry for that :)

muzixing commented 2 weeks ago

BTW, ,Med, I think you can revert back the modification back to the old text if do not agree, and merge the PR. We can submit this revision this week. We can continue the discussion and update accordingly, thanks!