2884 Commits (c508af3cff9a1abd85be4aa77d7aec31c92efd80)

Author	SHA1	Message	Date
Nicholas Bollweg	c508af3cff	add wasm mimetype	6 years ago
pgajdos	30a716fb85	remove redundant print statements	6 years ago
pgajdos	ec3c3dd89e	fix one more function	6 years ago
pgajdos	1c3dbcc6bf	do not use nose for testing	6 years ago
Kevin Bates	639da8d813	Add separate handler for /terminals/new	6 years ago
Kevin Bates	84ba421e86	Create auto-named terminals via /terminals/new	6 years ago
Kevin Bates	7778dc3202	Allow for redundant accesses of same endpoint	6 years ago
Kevin Bates	ac50d2e530	Add support for creating terminals via GET	6 years ago
Kevin Bates	1550410229	Merge pull request #5799 from mriedem/5798-json-logging ... Add support for JSON formatted logs	6 years ago
Kevin Bates	545dbab070	Merge pull request #5807 from rahulporuri/cln/remove-dead-code ... Remove code for unsupported python versions	6 years ago
Kevin Bates	18e4834b12	Merge pull request #5806 from rahulporuri/cln/update-super-usage ... Update super usage	6 years ago
Matt Riedemann	67ad5a1c06	Merge remote-tracking branch 'origin/master' into 5798-json-logging	6 years ago
Kevin Bates	158cc007e8	Merge pull request #5805 from rahulporuri/cln/remove-unnecessary-future-imports ... Remove unnecessary future imports	6 years ago
Matt Riedemann	6a883e0a1c	Fix NotebookAppJSONLoggingTests.test_log_json_enabled ... The intent of the test is to make sure that log_json defaults to True when JUPYTER_ENABLE_JSON_LOGGING is set to "true" so this makes the test more explicit and to pass when json-logging is not installed so that does not interfere due to validation.	6 years ago
Matt Riedemann	4ee4bb24f9	Use unittest assertion methods for improved logging	6 years ago
Matt Riedemann	08f8ccd1f8	Add log_json=True test for log_request ... Basic test just to make sure log_request handles log_json=True properly.	6 years ago
Matt Riedemann	093aee290f	Add some basic tests for log_json config ... Make sure the default value is handled properly based on environment variables. Also checks the validation code based on whether or not json_logging is imported.	6 years ago
Matt Riedemann	fac228567e	Fix logging in _validate_log_json ... Just use the app logger since that has the default LogFormatter logging until we change it to JSON.	6 years ago
Matt Riedemann	8b06db3eae	Address review comments	6 years ago
Sai Rahul Poruri	2641b64ada	CLN : Remove code for unsupported python versions ... specifically, code which was handling python 2.x and < 3.4 modified: notebook/base/zmqhandlers.py modified: notebook/services/contents/tests/test_manager.py modified: notebook/tests/test_serverextensions.py	6 years ago
Sai Rahul Poruri	667e0d7519	CLN : Remove unnecessary future imports ... specifically print_function and absolute_import future imports	6 years ago
Sai Rahul Poruri	84df5297d5	CLN : Update super usage ... This commit updates the super usage. Because Python 2 is not supported anymore, super usage can be updated such that super is called without any arguments in the default case where super is called with the class name and self. Note that all usage of super has not been updated - a few cases which smelled funny have been ignored.	6 years ago
Sai Rahul Poruri	9dd2af27de	CLN : Remove utf8 coding cookies ... On Python 3, the default source file encoding for Python files is utf-8 and because Python 2 is no longer supported, the utf8 coding cookies can be removed	6 years ago
Matt Riedemann	47e06b013b	Log request properties separately when log_json=True ... This updates the log_request method to get the request properties logged separately if `log_json=True`. This ends up looking like: ``` {"written_at": "2020-10-09T20:59:52.044Z", "written_ts": 1602277192044732000, "msg": "", "type": "log", "logger": "NotebookApp", "thread": "MainThread", "level": "WARNING", "module": "log", "line_no": 50, "status": 404, "method": "GET", "ip": "127.0.0.1", "uri": "/api/kernels/a6dd447b-282a-4e1c-ab65-eb340ad12965/channels?session_id=472907c27ed94e2cbfbae86501f7d159", "request_time": 8.63, "referer": "None"} ```	6 years ago
Matt Riedemann	3e32ce576c	Cleanup for log_json review	6 years ago
Matt Riedemann	6a74393f95	Pass NotebookApp logger to log_request ... In order to use the same logger from the notebook app we need to pass it through to the log_request function. To do that we need to use a partial function so the `log` kwarg is available. If not provided the default logger is the same as before, the tornado web access logger. This is necessary for the `log_json` settings to be used for both the notebook app logs and the web app request logging. There is still a todo needed here so that the individual request fields get formatted separately rather than dumped into the `msg` field.	6 years ago
Matt Riedemann	ca95490d7c	Add log_json config option with default and validation handling ... This adds the `NotebookApp.log_json` CLI option which defaults to False. The default value comes from the `JUPYTER_ENABLE_JSON_LOGGING` environment variable. A validation hook is added such that if json logging is enabled but the `json-logging` package is not installed, an error is logged and the effective `log_json` value is False. If enabled and the `json-logging` package is installed, the `init_logging` method will initialize the non-web [1] json logging formatter. [1] https://github.com/bobbui/json-logging-python#21-non-web-application-log	6 years ago
Matt Riedemann	b617d91052	Check for json log formatting in log_request ... This gets us closer to logging parts of the request as separate json fields but not all the way there. For example: ```json {"written_at": "2020-10-07T21:52:16.975Z", "written_ts": 1602107536975497000, "msg": "{'status': 404, 'method': 'GET', 'ip': '127.0.0.1', 'uri': '/nbextensions/widgets/notebook/js/extension.js', 'request_time': 1.9807815551757812, 'referer': 'http://localhost:8888/notebooks/test.ipynb'}", "type": "log", "logger": "NotebookApp", "thread": "MainThread", "level": "WARNING", "module": "log", "line_no": 54} ```	6 years ago
Matt Riedemann	1929fb53b8	Add support for JSON formatted logs ... This adds a new extra package dependency on `json-logging` and an environment variable, which when set to "true" regardless of case, will try to use the json-logging non-web app log formatter. If the `json-logging` package is not installed but the environment variable is True, something like this will be logged but it will not crash the application: ``` $ ENABLE_JSON_LOGGING=true jupyter notebook Unable to use json logging due to missing packages. Run "pip install notebook[json-logging]" to fix. Traceback (most recent call last): File "/home/osboxes/jupyter/notebook/notebook/notebookapp.py", line 144, in <module> import json_logging ModuleNotFoundError: No module named 'json_logging' ``` Initially I tried to add a new Bool config option to toggle this but the problem is (from my limited understanding of traitlets and tornado) is that `_log_formatter_cls` needs to be set early and trying to se the log formatter later in `init_logging` is too late - or at least I couldn't figure out a way to reset the log formatter dynamically (I tried calling `_log_format_changed` [1] but that didn't work). With this working you get logs like this: ``` {"written_at": "2020-10-07T21:10:51.606Z", "written_ts": 1602105051606265000, "msg": "404 GET /nbextensions/widgets/notebook/js/extension.js (127.0.0.1) 9.26ms referer=http://localhost:8888/notebooks/Untitled.ipynb", "type": "log", "logger": "NotebookApp", "thread": "MainThread", "level": "WARNING", "module": "log", "line_no": 49} {"written_at": "2020-10-07T21:10:54.443Z", "written_ts": 1602105054443309000, "msg": "Starting buffering for f260ddd7-938c-42d0-ac3b-455bea76694f:49f30b53fc4b4ec6a8f2fb748a171613", "type": "log", "logger": "NotebookApp", "thread": "MainThread", "level": "INFO", "module": "kernelmanager", "line_no": 222} {"written_at": "2020-10-07T21:10:54.446Z", "written_ts": 1602105054446264000, "msg": "Kernel shutdown: f260ddd7-938c-42d0-ac3b-455bea76694f", "type": "log", "logger": "NotebookApp", "thread": "MainThread", "level": "INFO", "module": "multikernelmanager", "line_no": 201} ``` An obvious improvement here would to be able to split the `msg` fields apart so that we can log things like response status code, request method, request URL, response_time_ms etc. That should be possible with the `JSONLogWebFormatter` from `json-logging` but when I tried using that I was getting errors from the library about a correlation id (which is based on a request header we don't use). The `json-logging` library supports several web frameworks like Flask but unfortunately does not have built in support for Tornado, but it does support custom formatters so that might be a follow up option to improve on this. [1] https://github.com/ipython/traitlets/blob/4.3.3/traitlets/config/application.py#L195 Closes #5798	6 years ago
Kevin Bates	2ba296039a	Merge pull request #5488 from toonijn/refactor-markdown ... Refactor code duplication of markdown renderers	6 years ago
Kevin Bates	466f0f3096	Merge pull request #5780 from bdbai/patch-2 ... Update translation for zh_CN	6 years ago
mueslo	cd91bd1ac8	Prevent leaking of handle in _check_pid_win32 ... credit to @garu57 fixes #4862	6 years ago
Tomas Hrnciar	63ee7ac809	Move requests_unixsocket import outside of fetch_url()	6 years ago
包布丁	30097e16d9	Update translation for zh_CN	6 years ago
Kevin Bates	a35b511991	Back to dev version	6 years ago
Kevin Bates	d8308e1380	Release 6.1.4	6 years ago
Pavel Panchekha	e6e05853d0	Update notebook.js (#5733)	6 years ago
包布丁	652e86c3cd	Remove unnecessary ellipses	6 years ago
包布丁	c84af49dfd	Remove ellipsis	6 years ago
包布丁	9e7c8515a8	Fix typo for Check in zh_CN	6 years ago
Zsailer	45a7616075	bug fix: remove double encoding in download files	6 years ago
Zsailer	31fc47aa77	revert change in page alignment	6 years ago
Kevin Bates	8ef7e0bb49	Back to dev version	6 years ago
Kevin Bates	2d8b28adc0	Release 6.1.3	6 years ago
Kyle Kelley	311a4c70e8	Title new buttons with label if action undefined	6 years ago
Kevin Bates	7ca3b22128	Back to dev version	6 years ago
Kevin Bates	38d8308382	Release 6.1.2	6 years ago
Zachary Sailer	0c2d119be4	Merge pull request #5672 from magdapoppins/tooltip-style-scope-fix ... Tooltip style scope fix	6 years ago
Zachary Sailer	498b0bcefb	Merge pull request #5666 from kevin-bates/bind-unix-socket ... Remove unnecessary import of bind_unix_socket	6 years ago
magda	22b40c0deb	Clean up by removing unused styles.	6 years ago