Clarifications on CLI and handlers

Clarifications on CLI and handlers

par Olivér Facklam,
Nombre de réponses : 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

En réponse à Olivér Facklam

Re: Clarifications on CLI and handlers

par 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

En réponse à Olivér Facklam

Re: Clarifications on CLI and handlers

par 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.

En réponse à Clement Luc Irenee Blaudeau

Re: Clarifications on CLI and handlers

par 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