Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Better server-side question-error handling #32

Closed
mwest1066 opened this issue May 22, 2014 · 1 comment
Closed

Better server-side question-error handling #32

mwest1066 opened this issue May 22, 2014 · 1 comment
Labels
enhancement A desired new feature or change (not a bug)

Comments

@mwest1066
Copy link
Member

Make sure server.js can survive and accurately report:

  1. Missing server.js for questions.
  2. Load/parse errors for server.js.
  3. Exceptions thrown from server.js functions.
  4. Invalid responses from server.js functions (extra data, missing data, mal-typed or mal-formed data (e.g., score < 0 or > 1)).
  5. Errors from dependent loads via r.js.

This was partially implemented in c3876b1bfa058afe65d3c23afcdc202419f69871 in old-PrairieLearn (at least the main server should survive question server errors).
@mwest1066 mwest1066 added this to the 1.19 - Code cleanup milestone Oct 15, 2015
@mwest1066 mwest1066 removed this from the Code cleanup milestone Feb 19, 2016
@mwest1066
Copy link
Member Author

Should now be fully implemented in PLv2.

mwest1066 pushed a commit that referenced this issue Oct 9, 2020
@dependabot-preview @mwest1066
[Security] Bump eslint-utils from 1.4.0 to 1.4.2 (#32)
9db487c 
Bumps [eslint-utils](https://github.com/mysticatea/eslint-utils) from 1.4.0 to 1.4.2. **This update includes a security fix.**
- [Release notes](https://github.com/mysticatea/eslint-utils/releases)
- [Commits](mysticatea/eslint-utils@v1.4.0...v1.4.2)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
frankyhao pushed a commit that referenced this issue Jan 19, 2021
@dependabot-preview
[Security] Bump eslint-utils from 1.4.0 to 1.4.2 (#32)
c226a2f 
Bumps [eslint-utils](https://github.com/mysticatea/eslint-utils) from 1.4.0 to 1.4.2. **This update includes a security fix.**
- [Release notes](https://github.com/mysticatea/eslint-utils/releases)
- [Commits](mysticatea/eslint-utils@v1.4.0...v1.4.2)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement A desired new feature or change (not a bug)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mwest1066