Skip to content

Add a new `extra-dispatcher-file' configuration option.#15

Closed
elibarzilay wants to merge 1 commit intoracket:masterfrom
elibarzilay:extra-dispatcher-file
Closed

Add a new `extra-dispatcher-file' configuration option.#15
elibarzilay wants to merge 1 commit intoracket:masterfrom
elibarzilay:extra-dispatcher-file

Conversation

@elibarzilay
Copy link
Member

This allows handling other requests implement additional course-related
functionality by the already running server. The documentation needs
more work though: it refers to `dispatcher/c' which needs to be added,
and there is no real sample code for it. (My use is too specific to be
used.)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"loaded"

This allows handling other requests implement additional course-related
functionality by the already running server.  The documentation needs
more work though: it refers to `dispatcher/c' which needs to be added,
and there is no real sample code for it.  (My use is too specific to be
used.)
@elibarzilay elibarzilay force-pushed the extra-dispatcher-file branch from 355c8f2 to 6c83721 Compare November 28, 2014 07:57
@samth
Copy link
Member

samth commented Sep 11, 2015

@elibarzilay Is this likely to be worked on in the future, or should I fix up the docs, or should it be closed?

@elibarzilay
Copy link
Member Author

I think that it's mostly fine, with really small things:

  • the typo that albertxing pointed at,
  • adding whatever require line to get the dispatcher/c binding,
  • the "example" could be better.

The reason for this addition is that if you want to handle other things,
you need to either add another separate web server (obviously bad), or
somehow wrap this one (complicated, since it can deal with both http and
the handin protocol).

@samth
Copy link
Member

samth commented Sep 11, 2015

@elibarzilay Will you make those changes, or do you want me to?

@elibarzilay
Copy link
Member Author

Superseded by #41.

(And better to merge as-is, and add the doc tweaks when needed in the future...?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants