Skip to content

Commit

Permalink
Merge pull request #622 from edx/feanil/fix_json_loading_for_python3
Browse files Browse the repository at this point in the history
Don't open for binary reading. We want to read the string into json.
  • Loading branch information
feanil authored Nov 18, 2019
2 parents 5a719a2 + 880e1a7 commit 95e9348
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion edx_proctoring/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,6 @@
from __future__ import absolute_import

# Be sure to update the version number in edx_proctoring/package.json
__version__ = '2.1.5'
__version__ = '2.1.6'

default_app_config = 'edx_proctoring.apps.EdxProctoringConfig' # pylint: disable=invalid-name
2 changes: 1 addition & 1 deletion edx_proctoring/apps.py
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ def make_worker_config(backends, out='/tmp/workers.json'):
try:
package = backend.npm_module
package_file = os.path.join(settings.NODE_MODULES_ROOT, package, 'package.json')
with open(package_file, 'rb') as package_fp:
with open(package_file, 'r') as package_fp:
package_json = json.load(package_fp)
main_file = package_json['main']
config[package] = ['babel-polyfill', os.path.join(settings.NODE_MODULES_ROOT, package, main_file)]
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"name": "@edx/edx-proctoring",
"//": "Be sure to update the version number in edx_proctoring/__init__.py",
"//": "Note that the version format is slightly different than that of the Python version when using prereleases.",
"version": "2.1.5",
"version": "2.1.6",
"main": "edx_proctoring/static/index.js",
"repository": {
"type": "git",
Expand Down

0 comments on commit 95e9348

Please sign in to comment.