mirror of
https://github.com/kuhyx/todo-app.git
synced 2026-07-04 17:03:04 +02:00
Krzysztof kuhy Rudnicki
0ce35ded4f
Connecting via the device flow saved the token to prefs but had no effect:
the capture screen cached its settings from launch and only re-adopted them
when Settings was popped *with a result*, which the connect flow never did.
So auto-sync kept using stale (token-less) settings (notes never downloaded)
and reopening Settings re-seeded the fields from the stale settings (empty
token → Test connection failed, Connect restarted every time).
- settings: on a successful connect, save then run a sync right away and
report the result ("Connected and synced …") so notes download and the
user gets real confirmation, instead of the inert "Token saved on Save".
- capture: always reload settings from storage after returning from Settings,
so a device-flow connect (which saves without popping a result) is picked up.
- tool/device_flow_check.dart: standalone end-to-end device-flow probe used to
confirm the OAuth App + token + repo-access chain is healthy (it is); the bug
was purely app-side token application.
152 tests, 100% line coverage.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
86 lines
2.9 KiB
Dart
86 lines
2.9 KiB
Dart
// Standalone end-to-end check of the GitHub OAuth device flow used by the app,
|
|
// to isolate whether a failure is in GitHub/OAuth-App config or in the app.
|
|
//
|
|
// Run: dart run tool/device_flow_check.dart
|
|
// It prints a user code, you authorize it in the browser, and it reports
|
|
// whether polling yields a token and whether that token can read the sync repo.
|
|
import 'dart:convert';
|
|
import 'dart:io';
|
|
|
|
import 'package:http/http.dart' as http;
|
|
|
|
const _clientId = 'Ov23li9tF2R46PqzJgch';
|
|
const _scope = 'repo';
|
|
const _owner = 'kuhyx';
|
|
const _repo = 'todo-sync';
|
|
|
|
Future<void> main() async {
|
|
final code = await http.post(
|
|
Uri.parse('https://github.com/login/device/code'),
|
|
headers: {'Accept': 'application/json'},
|
|
body: {'client_id': _clientId, 'scope': _scope},
|
|
);
|
|
if (code.statusCode != 200) {
|
|
stderr.writeln('device/code FAILED ${code.statusCode}: ${code.body}');
|
|
exit(1);
|
|
}
|
|
final dc = jsonDecode(code.body) as Map<String, dynamic>;
|
|
stdout.writeln(
|
|
'>>> Open ${dc['verification_uri']} and enter: ${dc['user_code']}',
|
|
);
|
|
stdout.writeln('Polling for the token...');
|
|
|
|
final deviceCode = dc['device_code'] as String;
|
|
var interval = (dc['interval'] as int?) ?? 5;
|
|
final deadline = DateTime.now().add(
|
|
Duration(seconds: (dc['expires_in'] as int?) ?? 900),
|
|
);
|
|
|
|
while (DateTime.now().isBefore(deadline)) {
|
|
await Future<void>.delayed(Duration(seconds: interval));
|
|
final res = await http.post(
|
|
Uri.parse('https://github.com/login/oauth/access_token'),
|
|
headers: {'Accept': 'application/json'},
|
|
body: {
|
|
'client_id': _clientId,
|
|
'device_code': deviceCode,
|
|
'grant_type': 'urn:ietf:params:oauth:grant-type:device_code',
|
|
},
|
|
);
|
|
final body = jsonDecode(res.body) as Map<String, dynamic>;
|
|
final token = body['access_token'] as String?;
|
|
if (token != null) {
|
|
stdout.writeln('TOKEN OK (length ${token.length})');
|
|
final repo = await http.get(
|
|
Uri.parse('https://api.github.com/repos/$_owner/$_repo'),
|
|
headers: {
|
|
'Authorization': 'Bearer $token',
|
|
'Accept': 'application/vnd.github+json',
|
|
'User-Agent': 'todo-app-sync',
|
|
},
|
|
);
|
|
stdout.writeln('repo $_owner/$_repo access status: ${repo.statusCode}');
|
|
stdout.writeln(
|
|
repo.statusCode == 200
|
|
? 'CHAIN OK — device flow + token + repo access all work.'
|
|
: 'TOKEN CANNOT READ REPO: ${repo.body}',
|
|
);
|
|
exit(repo.statusCode == 200 ? 0 : 2);
|
|
}
|
|
switch (body['error'] as String?) {
|
|
case 'authorization_pending':
|
|
stdout.write('.');
|
|
case 'slow_down':
|
|
interval = (body['interval'] as int?) ?? interval + 5;
|
|
case final String e:
|
|
stderr.writeln('\nTOKEN ERROR: $e — ${body['error_description']}');
|
|
exit(1);
|
|
case null:
|
|
stderr.writeln('\nUnexpected: ${res.body}');
|
|
exit(1);
|
|
}
|
|
}
|
|
stderr.writeln('\nExpired before authorization.');
|
|
exit(1);
|
|
}
|