Clarifications on CLI and handlers

Clarifications on CLI and handlers

by Olivér Facklam -
Number of replies: 3

Hello,

I have two questions:

1/ regarding the CLI, there is a note in the homework description stating the client should exit with a status code of 1 if a bad combination of parameters is given. However this doesn't seem to be implemented in the cli's main() function. Are we supposed to add this, or is it ok with the current implementation? Also should all CLI messages still go to "/message"?

2/ I noticed there is no handler defined for DataReply and SearchReply packet... can / should we add these in ourselves?

Thanks a lot, and have a nice weekend

Olivér

In reply to Olivér Facklam

Re: Clarifications on CLI and handlers

by Kirill Nikitin -

Hi,

1) It is ok with the current implementation. And yes, it should go to "/message"; you can see in controller.go that it expects the download and search-related requests there.

2) Yes, you should add / implement them. We probably forgot to add the placeholders...

Best,

Kirill

In reply to Olivér Facklam

Re: Clarifications on CLI and handlers

by Clement Luc Irenee Blaudeau -

I'm still a bit confused by what we are supposed to do with the CLI. Isn't already completed in the template? What changes to the sendMsg are we suppose to make ? I'm sorry if it is obvious but I've been reading again and again the handout and I'm still lost.

In reply to Clement Luc Irenee Blaudeau

Re: Clarifications on CLI and handlers

by Kirill Nikitin -

If your sendMsg was working correctly before, I believe you do not need to make any changes because the new cli additions reuse the same packet logic. Essentially, you just need to make sure that the cli commands from the handouts work.

Kirill