Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(37)

Unified Diff: utils/pub/oauth2.dart

Issue 11472016: Revert "Add logging system to pub and sprinkle some logging in." (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « utils/pub/log.dart ('k') | utils/pub/pub.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: utils/pub/oauth2.dart
diff --git a/utils/pub/oauth2.dart b/utils/pub/oauth2.dart
index a7800e4224bd02ae8eb0f74e38576848c28fcbfc..c7a87fdea7603274411a40b0f716f48f566f88e6 100644
--- a/utils/pub/oauth2.dart
+++ b/utils/pub/oauth2.dart
@@ -10,7 +10,6 @@ import 'dart:uri';
// TODO(nweiz): Make this a "package:" URL, or something nicer than this.
import '../../pkg/oauth2/lib/oauth2.dart';
import 'io.dart';
-import 'log.dart' as log;
import 'system_cache.dart';
import 'utils.dart';
@@ -101,24 +100,14 @@ Future<Client> _getClient(SystemCache cache) {
/// filesystem if possible. If the credentials can't be loaded for any reason,
/// the returned [Future] will complete to null.
Future<Credentials> _loadCredentials(SystemCache cache) {
- log.fine('Loading OAuth2 credentials.');
-
- if (_credentials != null) {
- log.fine('Using already-loaded credentials.');
- return new Future.immediate(_credentials);
- }
-
- var path = _credentialsFile(cache);
- return fileExists(path).chain((credentialsExist) {
- if (!credentialsExist) {
- log.fine('No credentials found at $path.');
- return new Future.immediate(null);
- }
+ if (_credentials != null) return new Future.immediate(_credentials);
+ return fileExists(_credentialsFile(cache)).chain((credentialsExist) {
+ if (!credentialsExist) return new Future.immediate(null);
return readTextFile(_credentialsFile(cache)).transform((credentialsJson) {
var credentials = new Credentials.fromJson(credentialsJson);
if (credentials.isExpired && !credentials.canRefresh) {
- log.error("Pub's authorization to upload packages has expired and "
+ printError("Pub's authorization to upload packages has expired and "
"can't be automatically refreshed.");
return null; // null means re-authorize
}
@@ -126,7 +115,8 @@ Future<Credentials> _loadCredentials(SystemCache cache) {
return credentials;
});
}).transformException((e) {
- log.error('Warning: could not load the saved OAuth2 credentials: $e\n'
+ printError('Warning: could not load the saved OAuth2 credentials:'
+ ' $e\n'
'Obtaining new credentials...');
return null; // null means re-authorize
});
@@ -135,11 +125,10 @@ Future<Credentials> _loadCredentials(SystemCache cache) {
/// Save the user's OAuth2 credentials to the in-memory cache and the
/// filesystem.
Future _saveCredentials(SystemCache cache, Credentials credentials) {
- log.fine('Saving OAuth2 credentials.');
_credentials = credentials;
- var path = _credentialsFile(cache);
- return ensureDir(dirname(path)).chain((_) =>
- writeTextFile(path, credentials.toJson()));
+ var credentialsFile = _credentialsFile(cache);
+ return ensureDir(dirname(credentialsFile)).chain((_) =>
+ writeTextFile(credentialsFile, credentials.toJson()));
}
/// The path to the file in which the user's OAuth2 credentials are stored.
@@ -172,7 +161,7 @@ Future<Client> _authorize() {
server.addRequestHandler((request) => request.path == "/",
(request, response) {
chainToCompleter(new Future.immediate(null).chain((_) {
- log.message('Authorization received, processing...');
+ print('Authorization received, processing...');
var queryString = request.queryString;
if (queryString == null) queryString = '';
response.statusCode = 302;
@@ -191,14 +180,13 @@ Future<Client> _authorize() {
var authUrl = grant.getAuthorizationUrl(
new Uri.fromString('http://localhost:${server.port}'), scopes: _scopes);
- log.message(
- 'Pub needs your authorization to upload packages on your behalf.\n'
- 'In a web browser, go to $authUrl\n'
- 'Then click "Allow access".\n\n'
- 'Waiting for your authorization...');
+ print('Pub needs your authorization to upload packages on your behalf.\n'
+ 'In a web browser, go to $authUrl\n'
+ 'Then click "Allow access".\n\n'
+ 'Waiting for your authorization...');
return completer.future.transform((client) {
- log.message('Successfully authorized.\n');
+ print('Successfully authorized.\n');
return client;
});
}
« no previous file with comments | « utils/pub/log.dart ('k') | utils/pub/pub.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698