SolidOS / solidos

The operating system for Solid
https://solidos.solidcommunity.net/
MIT License
127 stars 19 forks source link

No credentials sent for fetch of preferences file -> 401 -> alert #140

Closed timbl closed 2 years ago

timbl commented 2 years ago

HTTP status 401 for preference file https://timbl.inrupt.net/settings/prefs.ttl

Request headers:

Accept: */*
Accept-Encoding: gzip, deflate, br
Accept-Language: en-US,en;q=0.9,fr;q=0.8
Connection: keep-alive
Host: timbl.inrupt.net
Origin: https://timbl.com
Referer: https://timbl.com/
sec-ch-ua: " Not A;Brand";v="99", "Chromium";v="96", "Google Chrome";v="96"
sec-ch-ua-mobile: ?0
sec-ch-ua-platform: "macOS"
Sec-Fetch-Dest: empty
Sec-Fetch-Mode: cors
Sec-Fetch-Site: cross-site
sec-gpc: 1
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.110 Safari/537.36

Response headers:

Request URL: https://timbl.inrupt.net/settings/prefs.ttl
Request Method: GET
Status Code: 401 Unauthenticated
Remote Address: 35.168.136.65:443
Referrer Policy: strict-origin-when-cross-origin

stack:


  | (anonymous) | @ | mashlib.js:35951
-- | -- | -- | --
  | fetch | @ | mashlib.js:35888
  | fetchUri | @ | mashlib.js:10545
  | pendingFetchPromise | @ | mashlib.js:10392
  | load | @ | mashlib.js:10380
  | (anonymous) | @ | mashlib.js:10371
  | load | @ | mashlib.js:10370
  | SolidLogic.load | @ | mashlib.js:57799
  | (anonymous) | @ | mashlib.js:57835
  | step | @ | mashlib.js:57563
  | (anonymous) | @ | mashlib.js:57494
  | (anonymous) | @ | mashlib.js:57466
  | ../solid-logic/lib/logic/SolidLogic.js.__awaiter | @ | mashlib.js:57445
  | SolidLogic.loadIndexes | @ | mashlib.js:57817
  | (anonymous) | @ | mashlib.js:58716
  | step | @ | mashlib.js:58623
  | (anonymous) | @ | mashlib.js:58554
  | (anonymous) | @ | mashlib.js:58526
  | ../solid-logic/lib/typeIndex/typeIndexLogic.js.__awaiter | @ | mashlib.js:58505
  | loadIndex | @ | mashlib.js:58706
  | _callee6$ | @ | mashlib.js:137152
  | tryCatch | @ | mashlib.js:158909
  | invoke | @ | mashlib.js:159140
  | (anonymous) | @ | mashlib.js:158965
  | asyncGeneratorStep | @ | mashlib.js:153113
  | _next | @ | mashlib.js:153135
  | Promise.then (async) |   |  
  | asyncGeneratorStep | @ | mashlib.js:153123
  | _next | @ | mashlib.js:153135
  | (anonymous) | @ | mashlib.js:153142
  | (anonymous) | @ | mashlib.js:153131
  | findAppInstances | @ | mashlib.js:137100
  | _callee6$ | @ | mashlib.js:137124
  | tryCatch | @ | mashlib.js:158909
  | invoke | @ | mashlib.js:159140
  | (anonymous) | @ | mashlib.js:158965
  | asyncGeneratorStep | @ | mashlib.js:153113
  | _next | @ | mashlib.js:153135
  | (anonymous) | @ | mashlib.js:153142
  | (anonymous) | @ | mashlib.js:153131
  | _findAppInstances | @ | mashlib.js:137247
  | findAppInstances | @ | mashlib.js:137100
  | _callee6$ | @ | mashlib.js:72783
  | tryCatch | @ | mashlib.js:158909
  | invoke | @ | mashlib.js:159140
  | (anonymous) | @ | mashlib.js:158965
  | asyncGeneratorStep | @ | mashlib.js:72346
  | _next | @ | mashlib.js:72368
  | (anonymous) | @ | mashlib.js:72375
  | (anonymous) | @ | mashlib.js:72364
  | _getAddressBooks3 | @ | mashlib.js:72816
  | _getAddressBooks2 | @ | mashlib.js:72820
  | _callee7$ | @ | mashlib.js:73029
  | tryCatch | @ | mashlib.js:158909
  | invoke | @ | mashlib.js:159140
  | (anonymous) | @ | mashlib.js:158965
  | asyncGeneratorStep | @ | mashlib.js:72346
  | _next | @ | mashlib.js:72368
  | (anonymous) | @ | mashlib.js:72375
  | (anonymous) | @ | mashlib.js:72364
  | _getDashboardItems | @ | mashlib.js:73057
  | getDashboardItems | @ | mashlib.js:72762
  | _callee3$ | @ | mashlib.js:71475
  | tryCatch | @ | mashlib.js:158909
  | invoke | @ | mashlib.js:159140
  | (anonymous) | @ | mashlib.js:158965
  | asyncGeneratorStep | @ | mashlib.js:71310
  | _next | @ | mashlib.js:71332
  | (anonymous) | @ | mashlib.js:71339
  | (anonymous) | @ | mashlib.js:71328
  | _getMenuItems | @ | mashlib.js:71495
  | getMenuItems | @ | mashlib.js:71464
  | _callee2$ | @ | mashlib.js:71440
  | tryCatch | @ | mashlib.js:158909
  | invoke | @ | mashlib.js:159140
  | (anonymous) | @ | mashlib.js:158965
  | asyncGeneratorStep | @ | mashlib.js:71310
  | _next | @ | mashlib.js:71332
  | (anonymous) | @ | mashlib.js:71339
  | (anonymous) | @ | mashlib.js:71328
  | _setUserMenu | @ | mashlib.js:71455
  | setUserMenu | @ | mashlib.js:71414
  | _callee$ | @ | mashlib.js:71378
  | tryCatch | @ | mashlib.js:158909
  | invoke | @ | mashlib.js:159140
  | (anonymous) | @ | mashlib.js:158965
  | asyncGeneratorStep | @ | mashlib.js:71310
  | _next | @ | mashlib.js:71332
  | (anonymous) | @ | mashlib.js:71339
  | (anonymous) | @ | mashlib.js:71328
  | _createHeader | @ | mashlib.js:71392
  | createHeader | @ | mashlib.js:71366
  | _callee$ | @ | mashlib.js:71598
  | tryCatch | @ | mashlib.js:158909
  | invoke | @ | mashlib.js:159140
  | (anonymous) | @ | mashlib.js:158965
  | asyncGeneratorStep | @ | mashlib.js:71547
  | _next | @ | mashlib.js:71569
  | (anonymous) | @ | mashlib.js:71576
  | (anonymous) | @ | mashlib.js:71565
  | _initMainPage | @ | mashlib.js:71612
  | initMainPage | @ | mashlib.js:71582
  | (anonymous) | @ | mashlib.js:164521
timbl commented 2 years ago

Its as though the new fetch which we get when we log in never gets slotted in at all ... The fetch at mashlib.js:35888 in the stack trace is a bit of JS which calls XMLHTTPRequest. Where exactly does that switch happen I wonder..

timbl commented 2 years ago

We bug in solidLogic crep in from cleaning up variable names . Should be fixed by https://github.com/solid/solid-logic/pull/35