Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: make it webpackable #371

Merged
merged 87 commits into from
Jan 8, 2019
Merged
Changes from 1 commit
Commits
Show all changes
87 commits
Select commit Hold shift + click to select a range
abba256
try to pass JSONP callback for revoke call
alexander-fenster Apr 25, 2018
71c2a73
fix test for revoke JSONP as well
alexander-fenster Apr 25, 2018
627718c
a method to get revoke URL
alexander-fenster Apr 25, 2018
271ca2e
config tweaks
alexander-fenster Apr 27, 2018
06d2006
feat: browser-compatible auth
alexander-fenster Jun 1, 2018
4fb8fbd
Merge branch 'master' into webbbpackkk
alexander-fenster Jun 1, 2018
a43775b
make tests pass
alexander-fenster Jun 1, 2018
df8737a
remove logging from test
alexander-fenster Jun 1, 2018
2530fd7
make it work
alexander-fenster Jun 1, 2018
d6a9193
resolve conflicts
alexander-fenster Jun 2, 2018
f50c130
switch looks nicer
alexander-fenster Jun 2, 2018
2240281
we need just one file
alexander-fenster Jun 2, 2018
ee3b6cb
pr feedback
alexander-fenster Jun 2, 2018
44bfb70
async/await ftw
alexander-fenster Jun 2, 2018
c961706
nit
alexander-fenster Jun 2, 2018
f5d13cf
no user-agent for browser
alexander-fenster Jun 2, 2018
66cfd8b
make sure we have crypto.subtle
alexander-fenster Jun 4, 2018
fc22478
merge merge merge
alexander-fenster Jun 4, 2018
789877d
test webpacking
alexander-fenster Jun 4, 2018
3e04387
timeouts++
alexander-fenster Jun 4, 2018
9171931
4 min to npm install o_O
alexander-fenster Jun 4, 2018
1fe2da8
node4 does not have npx
alexander-fenster Jun 4, 2018
9cfc3d1
node6 does not have npx
alexander-fenster Jun 4, 2018
887037d
no webpack on node4
alexander-fenster Jun 4, 2018
e21f4f2
no webpack on node4 and node6
alexander-fenster Jun 4, 2018
4d69ef2
timeout for cleanup
alexander-fenster Jun 4, 2018
808f9f9
timeout for kitchen
alexander-fenster Jun 4, 2018
b67ef44
try this
alexander-fenster Jun 4, 2018
e1d29b1
try this
alexander-fenster Jun 4, 2018
94591de
try this
alexander-fenster Jun 4, 2018
b9e0b6f
i will always run tests locally before I push
alexander-fenster Jun 4, 2018
85e50fc
skip tests properly
alexander-fenster Jun 5, 2018
35141c6
more stuff
alexander-fenster Nov 29, 2018
b75afe5
Merge branch 'master' into webbbpackkk
alexander-fenster Dec 5, 2018
18c9194
compiles but tests still fail
alexander-fenster Dec 6, 2018
4ebfda2
Merge branch 'master' into webbbpackkk
alexander-fenster Dec 13, 2018
1f29d4d
tests pass now
alexander-fenster Dec 14, 2018
e50b5c2
use fast-text-encoding instead of text-encoding-shim
alexander-fenster Dec 14, 2018
824b225
Merge branch 'master' into webbbpackkk
alexander-fenster Dec 14, 2018
fb40045
fixing webpack config
alexander-fenster Dec 14, 2018
a26c530
Merge branch 'webbbpackkk' of github.com:google/google-auth-library-n…
alexander-fenster Dec 14, 2018
4a9ce97
system tests
alexander-fenster Dec 14, 2018
7468fea
remove unused fixture
alexander-fenster Dec 14, 2018
ad46838
Merge branch 'master' into webbbpackkk
alexander-fenster Dec 17, 2018
599969e
no pemverifier
alexander-fenster Dec 17, 2018
9c4de7a
avoid using crypto in googleauth.ts
alexander-fenster Dec 18, 2018
758e45b
gts fix
alexander-fenster Dec 18, 2018
e43f789
formatting package.json
alexander-fenster Dec 18, 2018
3d67ece
more gts fix and comments
alexander-fenster Dec 18, 2018
125bc82
never enough of gts fix
alexander-fenster Dec 19, 2018
47b4ae6
fix(deps): upgrade to gcp-metadata v0.9.0 (#536)
JustinBeckwith Dec 20, 2018
6f96525
Merge branch 'next' into webbbpackkk
alexander-fenster Dec 20, 2018
c6341e1
Merge branch 'master' into webbbpackkk
alexander-fenster Dec 27, 2018
27e59ac
browser tests with karma: initial commit
alexander-fenster Dec 29, 2018
504591a
test fixed
alexander-fenster Dec 29, 2018
5528473
prettier
alexander-fenster Dec 29, 2018
e094fd0
Merge branch 'master' into webbbpackkk
alexander-fenster Jan 3, 2019
f830859
make system tests work again
alexander-fenster Jan 3, 2019
09245be
more tests and cleaner interface
alexander-fenster Jan 3, 2019
af664bd
more tests and cleaner interface
alexander-fenster Jan 3, 2019
4fed7fb
gts fix
alexander-fenster Jan 3, 2019
c1d2da8
some careful merging
alexander-fenster Jan 4, 2019
78b1582
deprecate with style
alexander-fenster Jan 4, 2019
4e2f5bd
Merge branch 'master' into webbbpackkk
jkwlui Jan 4, 2019
436f8c4
final countdown
alexander-fenster Jan 4, 2019
a4bce49
Merge branch 'webbbpackkk' of github.com:google/google-auth-library-n…
alexander-fenster Jan 4, 2019
5c6e09b
owwww gts fix again
alexander-fenster Jan 4, 2019
b3e14eb
copyright 2019
alexander-fenster Jan 4, 2019
63accd1
use special docker image; --no-sandbox under Docker
jkwlui Jan 4, 2019
31f937a
Merge branch 'webbbpackkk' of github.com:google/google-auth-library-n…
jkwlui Jan 4, 2019
bfe1ea0
no semver
alexander-fenster Jan 4, 2019
999c075
Merge branch 'webbbpackkk' of github.com:google/google-auth-library-n…
alexander-fenster Jan 4, 2019
07809c7
Merge branch 'master' into webbbpackkk
alexander-fenster Jan 4, 2019
7e9b483
pr feedback
alexander-fenster Jan 4, 2019
63fcb02
ts-ignore TS2345 for sinon
alexander-fenster Jan 4, 2019
e5753a1
ts-ignore TS2345 for sinon
alexander-fenster Jan 4, 2019
049f2b8
pr feedback
alexander-fenster Jan 4, 2019
dfef932
arghhhh
alexander-fenster Jan 4, 2019
e570af4
reduce usage of util and Buffer
alexander-fenster Jan 5, 2019
d43bca0
pr feedback
alexander-fenster Jan 7, 2019
d87e003
try without ts-ignore
alexander-fenster Jan 7, 2019
53900c6
sinon vs typings
alexander-fenster Jan 7, 2019
32a70bd
require fast-text-encoding only if needed
alexander-fenster Jan 7, 2019
902e16c
no URL in webpack
alexander-fenster Jan 7, 2019
e1b0b3c
gts fix
alexander-fenster Jan 7, 2019
758e458
using window.URL in browser tests
alexander-fenster Jan 7, 2019
d2da190
gts fix
alexander-fenster Jan 7, 2019
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
22 changes: 9 additions & 13 deletions browser-test/test.oauth2.ts
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ import * as fs from 'fs';
import * as path from 'path';
import * as qs from 'querystring';
import * as sinon from 'sinon';
import * as url from 'url';

import {BodyResponseCallback} from '../src/transporters';

Expand Down Expand Up @@ -111,25 +110,22 @@ describe('Browser OAuth2 tests', () => {
});
});

it('should generate a valid consent page url', done => {
it('should generate a valid consent page url', () => {
const opts = {
access_type: ACCESS_TYPE,
scope: SCOPE,
response_type: 'code token'
};

const generated = client.generateAuthUrl(opts);
const parsed = url.parse(generated);
if (typeof parsed.query !== 'string') {
throw new Error('Unable to parse querystring');
}
const query = qs.parse(parsed.query);
// can't use URL class in webpack, so parsing URL manually
const queryString = generated.replace(/^.*?\?(.*?)(?:#.*)?$/, "$1");
alexander-fenster marked this conversation as resolved.
Show resolved Hide resolved
const query = qs.parse(queryString);
assert.strictEqual(query.response_type, 'code token');
assert.strictEqual(query.access_type, ACCESS_TYPE);
assert.strictEqual(query.scope, SCOPE);
assert.strictEqual(query.client_id, CLIENT_ID);
assert.strictEqual(query.redirect_uri, REDIRECT_URI);
done();
});

it('getToken should work', async () => {
Expand Down Expand Up @@ -179,11 +175,11 @@ describe('Browser OAuth2 tests', () => {
code_challenge: codes.codeChallenge,
code_challenge_method: CodeChallengeMethod.S256
});
const parsed = url.parse(authUrl);
assert.strictEqual(typeof parsed.query, 'string');
const props = qs.parse(parsed.query as string);
assert.strictEqual(props.code_challenge, codes.codeChallenge);
assert.strictEqual(props.code_challenge_method, CodeChallengeMethod.S256);
// can't use URL class in webpack, so parsing URL manually
const queryString = authUrl.replace(/^.*?\?(.*?)(?:#.*)?$/, "$1");
const query = qs.parse(queryString);
assert.strictEqual(query.code_challenge, codes.codeChallenge);
assert.strictEqual(query.code_challenge_method, CodeChallengeMethod.S256);
});

it('should verify a valid certificate against a jwt', async () => {
Expand Down