Skip to content
This repository was archived by the owner on Aug 1, 2024. It is now read-only.

Commit 39fa404

Browse files
fix: fix insights port typo in provision script
There was a typo in the port variable in the Insights provision script. It was 18011, but it should have been 18110. 18110 is what is used in the port mapping for the Docker container. This bug was observed when developers were unable to login to Insights. This was due to a mismatch in the redirect URIs as configured in the LMS's insights-sso Django OAuth Toolkit Application, which uses the port variable in the provision script. The actual redirect URI being used was using port 18110. The oauth2 standard requires strict equality of the redirect_uris, causing a failure.
1 parent 07211ea commit 39fa404

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

provision-insights.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ set -eu -o pipefail
55
set -x
66

77
name=insights
8-
port=18011
8+
port=18110
99

1010
docker-compose up -d insights
1111

0 commit comments

Comments
 (0)