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(Live): Display more info in the console #1311

Merged
merged 1 commit into from
Jun 12, 2024
Merged

feat(Live): Display more info in the console #1311

merged 1 commit into from
Jun 12, 2024

Conversation

lslezak
Copy link
Contributor

@lslezak lslezak commented Jun 11, 2024

Problem

  • Similar to printing the root password to the console the users should know more details about the system, esp.security related data like the Agama SSL certificate fingerprint

Solution

  • Add several issue generators which contain the required details

Displayed Info

  • Agama welcome message, it would be nice to welcome the users 🤝
  • The welcome message contains the Agama version (8+227 means version 8 + additional 227 commits from Git, for the public release it will display only the plain version like 9)
  • SSH host key fingerprints, useful when connecting via SSH to verify that you are really connecting to the right machine
  • Agama SSL certificate fingerprint, when connecting remotely you should verify that the same fingerprint is reported by the browser
  • Prints Agama URLs for connection, including the mDNS name
  • (The generated root password was already printed at the console before.)

Details

  • The mDNS name (e.g. "agama.local") is directly obtained from the Avahi server, if you start a second Agama instance it will use the "agama-2.local" name to avoid conflicts. If the Avahi daemon does not run or is stopped then the item is removed from the list.
  • Also the SSL certificate fingerprint is automatically refreshed whenever the certificate file is updated or removed.
  • Using the udev rule ensures that the dynamically added/removed network devices will be handled correctly
  • The version in the welcome message is static (not refreshed after creating it), later when we support some kind of self-update we could possibly regenerate it as well
  • The same applies for the SSH host key fingerprints , currently they are static

Testing

  • Tested manually

Screenshots

agama_console_messages

Copy link
Contributor

@dgdavid dgdavid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code-wise I can't help too much, but the overall approach LGTM.

Thanks for such a cool and complete welcome! ❤️

Copy link
Contributor

@imobachgs imobachgs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! Thanks a lot.

@@ -0,0 +1,5 @@
# udev rules for generating the Agama access URLs displayed at the console
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice trick :-)

@lslezak lslezak merged commit 1ae4908 into master Jun 12, 2024
@lslezak lslezak deleted the console_info branch June 12, 2024 07:06
@imobachgs imobachgs mentioned this pull request Jun 27, 2024
imobachgs added a commit that referenced this pull request Jun 27, 2024
Prepare for releasing Agama 9. It includes the following pull requests:

- #1101
- #1202
- #1228
- #1231
- #1236
- #1238
- #1239
- #1240
- #1242
- #1243
- #1244
- #1245
- #1246
- #1247
- #1248
- #1249
- #1250
- #1251
- #1252
- #1253
- #1254
- #1255
- #1256
- #1257
- #1258
- #1259
- #1260
- #1261
- #1264
- #1265
- #1267
- #1268
- #1269
- #1270
- #1271
- #1272
- #1273
- #1274
- #1279
- #1280
- #1284
- #1285
- #1286
- #1287
- #1288
- #1289
- #1290
- #1291
- #1292
- #1293
- #1294
- #1295
- #1296
- #1298
- #1299
- #1300
- #1301
- #1302
- #1303
- #1304
- #1305
- #1306
- #1307
- #1308
- #1309
- #1310
- #1311
- #1312
- #1313
- #1314
- #1315
- #1316
- #1317
- #1318
- #1319
- #1320
- #1321
- #1322
- #1323
- #1324
- #1325
- #1326
- #1328
- #1329
- #1331
- #1332
- #1334
- #1338
- #1340
- #1341
- #1342
- #1343
- #1344
- #1345
- #1348
- #1349
- #1351
- #1352
- #1353
- #1354
- #1355
- #1356
- #1357
- #1358
- #1359
- #1360
- #1361
- #1362
- #1363
- #1365
- #1366
- #1367
- #1368
- #1371
- #1372
- #1374
- #1375
- #1376
- #1379
- #1380
- #1381
- #1383
- #1384
- #1385
- #1386
- #1387
- #1388
- #1389
- #1391
- #1392
- #1394
- #1395
- #1397
- #1398
- #1399
- #1400
- #1403
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants