keeweb
/
keeweb
mirror of https://github.com/keeweb/keeweb.git
1
0
Fork 0
Code Issues Releases Wiki Activity

Merge branch 'master' into develop 

# Conflicts:
#	package.json
#	release-notes.md
This commit is contained in:
antelle 2020-05-04 19:45:03 +02:00
parent f031be4ce2 03c44d76c9
commit 316518d63b
No known key found for this signature in database
GPG Key ID: 094A2F2D6136A4EE
15 changed files with 888 additions and 742 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
Gruntfile.js
View File

@ -61,14 +61,11 @@ module.exports = function(grunt) {
dest: 'tmp/index.html',
nonull: true
},
'html-dist': {
src: 'tmp/app.html',
dest: 'dist/index.html',
nonull: true
},
'404-dist': {
src: 'app/404.html',
dest: 'dist/404.html',
'content-dist': {
cwd: 'app/content/',
src: '**',
dest: 'dist/',
expand: true,
nonull: true
},
favicon: {
@ -194,7 +191,7 @@ module.exports = function(grunt) {
},
app: {
src: 'tmp/app.html',
dest: 'tmp/app.html'
dest: 'dist/index.html'
}
},
htmlmin: {
@ -270,7 +267,10 @@ module.exports = function(grunt) {
sha: 'dev'
}),
publicPath: '/',
contentBase: path.resolve(__dirname, 'tmp'),
contentBase: [
path.resolve(__dirname, 'tmp'),
path.resolve(__dirname, 'app/content')
],
progress: false
},
js: {

0
app/404.html → app/content/404.html
View File

16
app/content/oauth-result/dropbox.html Normal file
View File

@ -0,0 +1,16 @@
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="UTF-8" />
<title>KeeWeb</title>
</head>
<body>
<script>
window.opener.postMessage(
{ storage: 'dropbox', search: location.search },
window.location.origin
);
window.close();
</script>
</body>
</html>

16
app/content/oauth-result/gdrive.html Normal file
View File

@ -0,0 +1,16 @@
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="UTF-8" />
<title>KeeWeb</title>
</head>
<body>
<script>
window.opener.postMessage(
{ storage: 'gdrive', search: location.search },
window.location.origin
);
window.close();
</script>
</body>
</html>

16
app/content/oauth-result/onedrive.html Normal file
View File

@ -0,0 +1,16 @@
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="UTF-8" />
<title>KeeWeb</title>
</head>
<body>
<script>
window.opener.postMessage(
{ storage: 'onedrive', search: location.search },
window.location.origin
);
window.close();
</script>
</body>
</html>

6
app/scripts/app.js
View File

@ -5,7 +5,6 @@ import { AppRightsChecker } from 'comp/app/app-rights-checker';
import { ExportApi } from 'comp/app/export-api';
import { SingleInstanceChecker } from 'comp/app/single-instance-checker';
import { Updater } from 'comp/app/updater';
import { AuthReceiver } from 'comp/browser/auth-receiver';
import { FeatureTester } from 'comp/browser/feature-tester';
import { FocusDetector } from 'comp/browser/focus-detector';
import { IdleTracker } from 'comp/browser/idle-tracker';
@ -33,11 +32,6 @@ const ready = (Launcher && Launcher.ready) || $;
ready(() => {
StartProfiler.milestone('document ready');
if (AuthReceiver.receive()) {
return;
}
StartProfiler.milestone('checking auth');
const appModel = new AppModel();
StartProfiler.milestone('creating app model');

31
app/scripts/comp/browser/auth-receiver.js
View File

@ -1,31 +0,0 @@
import { Features } from 'util/features';
const AuthReceiver = {
receive() {
if (!Features.isPopup) {
return false;
}
const opener = window.opener || window.parent;
const message = this.urlArgsToMessage(window.location.href);
const hasKeys = Object.keys(message).filter(key => key !== 'config').length > 0;
if (!hasKeys) {
return false;
}
opener.postMessage(message, window.location.origin);
window.close();
return true;
},
urlArgsToMessage(url) {
const message = {};
url.split(/[?#&]/g).forEach(part => {
const parts = part.split('=');
if (parts.length === 2) {
message[parts[0]] = decodeURIComponent(parts[1]);
}
});
return message;
}
};
export { AuthReceiver };

29
app/scripts/storage/storage-base.js
View File

@ -261,8 +261,7 @@ class StorageBase {
if (redirectUrl.lastIndexOf('file:', 0) === 0) {
redirectUrl = Links.WebApp;
}
redirectUrl = redirectUrl.split('?')[0];
return redirectUrl;
return new URL(`oauth-result/${this.name}.html`, redirectUrl).href;
}
_oauthAuthorize(callback) {
@ -284,7 +283,7 @@ class StorageBase {
let listener;
if (Features.isDesktop) {
listener = StorageOAuthListener.listen();
listener = StorageOAuthListener.listen(this.name);
session.redirectUri = listener.redirectUri;
} else {
session.redirectUri = this._getOauthRedirectUrl();
@ -334,15 +333,27 @@ class StorageBase {
if (e.origin !== location.origin) {
return;
}
if (e.data && e.data.error) {
this.logger.error('OAuth error', e.data.error, e.data.error_description);
callback('OAuth: ' + e.data.error);
} else if (e.data && e.data.code) {
if (!e.data || !e.data.storage || !e.data.search) {
this.logger.debug('Skipped empty OAuth message', e.data);
return;
}
if (e.data.storage !== this.name) {
this.logger.debug('Skipped OAuth message for another storage', e.data.storage);
return;
}
const data = {};
for (const [key, value] of new URLSearchParams(e.data.search).entries()) {
data[key] = value;
}
if (data.error) {
this.logger.error('OAuth error', data.error, data.error_description);
callback('OAuth: ' + data.error);
} else if (data.code) {
Events.off('popup-closed', popupClosed);
window.removeEventListener('message', windowMessage);
this._oauthCodeReceived(e.data, session, callback);
this._oauthCodeReceived(data, session, callback);
} else {
this.logger.debug('Skipped OAuth message', e.data);
this.logger.debug('Skipped OAuth message', data);
}
};
Events.on('popup-closed', popupClosed);

18
app/scripts/storage/storage-oauth-listener.js
View File

@ -9,7 +9,7 @@ const logger = new Logger('storage-oauth-listener');
const StorageOAuthListener = {
server: null,
listen() {
listen(storageName) {
if (this.server) {
this.stop();
}
@ -20,12 +20,17 @@ const StorageOAuthListener = {
});
const http = Launcher.req('http');
let resultHandled = false;
const server = http.createServer((req, resp) => {
resp.writeHead(200, 'OK', {
'Content-Type': 'text/plain; charset=UTF-8'
});
resp.end(Locale.appBrowserAuthComplete);
this.handleResult(req.url, listener);
if (!resultHandled) {
this.stop();
this.handleResult(req.url, listener);
resultHandled = true;
}
});
const port = DefaultPort;
@ -43,7 +48,7 @@ const StorageOAuthListener = {
listener.emit('ready');
});
listener.redirectUri = `http://localhost:${port}/oauth-result`;
listener.redirectUri = `http://localhost:${port}/oauth-result/${storageName}.html`;
return listener;
},
@ -55,9 +60,12 @@ const StorageOAuthListener = {
},
handleResult(url, listener) {
url = new URL(url, listener.redirectUri);
if (url.origin + url.pathname !== listener.redirectUri) {
logger.info('Skipped result', url, listener.redirectUri);
return;
}
logger.info('OAuth result with code received');
this.stop();
url = new URL(url, 'http://localhost');
const state = url.searchParams.get('state');
const code = url.searchParams.get('code');
listener.emit('result', { state, code });

2
desktop/package-lock.json generated
View File

@ -1,6 +1,6 @@
{
"name": "KeeWeb",
"version": "1.14.1",
"version": "1.14.2",
"lockfileVersion": 1,
"requires": true,
"dependencies": {

2
desktop/package.json
View File

@ -1,6 +1,6 @@
{
"name": "KeeWeb",
"version": "1.14.1",
"version": "1.14.2",
"description": "Free cross-platform password manager compatible with KeePass",
"main": "main.js",
"homepage": "https://keeweb.info",

3
grunt.tasks.js
View File

@ -12,8 +12,7 @@ module.exports = function(grunt) {
'inline',
'htmlmin',
'csp-hashes',
'copy:html-dist',
'copy:404-dist',
'copy:content-dist',
'string-replace:service-worker',
'string-replace:manifest',
'copy:dist-icons',

1438
package-lock.json generated
View File

File diff suppressed because it is too large Load Diff

30
package.json
View File

@ -1,6 +1,6 @@
{
"name": "keeweb",
"version": "1.14.1",
"version": "1.14.2",
"description": "Free cross-platform password manager compatible with KeePass",
"main": "Gruntfile.js",
"private": true,
@ -10,10 +10,10 @@
"url": "https://github.com/keeweb/keeweb"
},
"dependencies": {
"@babel/core": "^7.9.0",
"@babel/core": "^7.9.6",
"@babel/plugin-external-helpers": "^7.8.3",
"@babel/plugin-proposal-class-properties": "^7.8.3",
"@babel/preset-env": "^7.9.5",
"@babel/preset-env": "^7.9.6",
"@keeweb/keeweb-native-modules": "https://github.com/keeweb/keeweb-native-modules/releases/download/0.1.6/keeweb-native-modules.tgz",
"adm-zip": "^0.4.14",
"argon2-browser": "1.13.0",
@ -27,11 +27,11 @@
"bourbon": "^7.0.0",
"chai": "^4.2.0",
"cross-env": "^7.0.2",
"dompurify": "^2.0.8",
"electron": "^8.2.3",
"electron-builder": "^22.5.1",
"dompurify": "^2.0.10",
"electron": "^8.2.5",
"electron-builder": "^22.6.0",
"eslint": "^6.8.0",
"eslint-config-prettier": "^6.10.1",
"eslint-config-prettier": "^6.11.0",
"eslint-config-standard": "^14.1.1",
"eslint-plugin-import": "^2.20.2",
"eslint-plugin-node": "^11.1.0",
@ -65,12 +65,12 @@
"kdbxweb": "1.6.0",
"load-grunt-tasks": "5.1.0",
"lodash": "^4.17.15",
"marked": "^0.8.2",
"marked": "^1.0.0",
"mini-css-extract-plugin": "^0.9.0",
"mocha": "^7.1.1",
"morphdom": "^2.5.12",
"node-sass": "^4.13.1",
"node-stream-zip": "1.9.1",
"mocha": "^7.1.2",
"morphdom": "^2.6.0",
"node-sass": "^4.14.0",
"node-stream-zip": "1.10.1",
"normalize.css": "8.0.1",
"optimize-css-assets-webpack-plugin": "^5.0.3",
"pikaday": "1.8.0",
@ -85,16 +85,16 @@
"string-replace-webpack-plugin": "0.1.3",
"strip-sourcemap-loader": "0.0.1",
"sumchecker": "^3.0.1",
"terser-webpack-plugin": "^2.3.5",
"terser-webpack-plugin": "^3.0.0",
"time-grunt": "2.0.0",
"url-loader": "^4.1.0",
"webpack": "^4.42.1",
"webpack": "^4.43.0",
"webpack-bundle-analyzer": "^3.7.0",
"webpack-dev-server": "^3.10.3"
},
"optionalDependencies": {
"grunt-appdmg": "github:keeweb/grunt-appdmg#874ad83",
"keytar": "^5.5.0"
"keytar": "^5.6.0"
},
"scripts": {
"start": "grunt",

3
release-notes.md
View File

@ -8,6 +8,9 @@ Release notes
`+` #1350: clearing master password after auto lock period
`-` fix #1463: copying the original url instead of adding https:
##### v1.14.2 (2020-05-04)
`-` distinct redirect URIs for storage providers
##### v1.14.1 (2020-05-02)
`-` fix #1478: fixed proxy issues with storage providers