Skip to content

Commit

Permalink
move firebase auth handling to core (#151)
Browse files Browse the repository at this point in the history
* move firebase auth handling to core

CPCN-206
  • Loading branch information
petrjasek authored Nov 24, 2023
1 parent cc4574d commit 2f8742c
Show file tree
Hide file tree
Showing 15 changed files with 23 additions and 17,129 deletions.
2 changes: 1 addition & 1 deletion client/.npmrc
Original file line number Diff line number Diff line change
@@ -1 +1 @@
package-lock=true
package-lock=false
16,764 changes: 0 additions & 16,764 deletions client/package-lock.json

This file was deleted.

3 changes: 1 addition & 2 deletions client/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,7 @@
"build": "export NODE_ENV=production && webpack --progress --profile --colors"
},
"dependencies": {
"firebase": "9.6.11",
"newsroom-core": "github:superdesk/newsroom-core#release/2.5"
"newsroom-core": "github:superdesk/newsroom-core#develop"
},
"devDependencies": {
"babel-plugin-transform-object-rest-spread": "^6.26.0",
Expand Down
23 changes: 0 additions & 23 deletions client/src/auth.js

This file was deleted.

51 changes: 0 additions & 51 deletions client/src/login.js

This file was deleted.

42 changes: 0 additions & 42 deletions client/src/reset-password.js

This file was deleted.

8 changes: 1 addition & 7 deletions client/webpack.config.js
Original file line number Diff line number Diff line change
@@ -1,7 +1 @@
const path = require('path');
const config = require('newsroom-core/webpack.config');

config.entry.login = path.resolve(__dirname, 'src', 'login.js');
config.entry.reset_password = path.resolve(__dirname, 'src', 'reset-password.js');

module.exports = config;
module.exports = require('newsroom-core/webpack.config')
103 changes: 0 additions & 103 deletions server/cp/auth.py

This file was deleted.

6 changes: 3 additions & 3 deletions server/cp/common_settings.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,10 @@

AUTH_PROVIDERS.extend([{
"_id": "gip",
"name": lazy_gettext("Google"),
"auth_type": AuthProviderType.GOOGLE_OAUTH.value,
"name": lazy_gettext("Firebase"),
"auth_type": AuthProviderType.FIREBASE,
}, {
"_id": "azure",
"name": lazy_gettext("Azure"),
"auth_type": AuthProviderType.SAML.value,
"auth_type": AuthProviderType.SAML,
}])
3 changes: 1 addition & 2 deletions server/requirements.in
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
google-auth==2.6.0
git+https://github.com/superdesk/newsroom-core.git@release/2.5#egg=Newsroom-Core
git+https://github.com/superdesk/newsroom-core.git@develop#egg=Newsroom-Core
4 changes: 2 additions & 2 deletions server/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -161,7 +161,7 @@ flask-wtf==0.14.3
future==0.18.3
# via python-twitter
google-auth==2.6.0
# via -r requirements.in
# via newsroom-core
gunicorn==20.0.4
# via newsroom-core
hachoir==3.0a3
Expand Down Expand Up @@ -225,7 +225,7 @@ mongolock==1.3.4
# via superdesk-core
natsort==8.4.0
# via croniter
newsroom-core @ git+https://github.com/superdesk/newsroom-core.git@release/2.5
newsroom-core @ git+https://github.com/superdesk/newsroom-core.git@develop
# via -r requirements.in
oauth2client==4.1.3
# via flask-oidc-ex
Expand Down
2 changes: 0 additions & 2 deletions server/settings.py
Original file line number Diff line number Diff line change
Expand Up @@ -208,7 +208,6 @@


BLUEPRINTS = [
"cp.auth", # we need this one loaded before newsroom.auth to make it override logout
"cp.mgmt_api_docs",
] + [
blueprint
Expand All @@ -233,7 +232,6 @@
INSTALLED_APPS = [
"cp.sidenav",
"cp.signals",
"cp.auth",
"newsroom.auth.saml",
]

Expand Down
38 changes: 0 additions & 38 deletions server/theme/cp_reset_password.html

This file was deleted.

25 changes: 0 additions & 25 deletions server/theme/cp_reset_password_confirmation.html

This file was deleted.

Loading

0 comments on commit 2f8742c

Please sign in to comment.