Andrey

Results 70 comments of Andrey

DevTools screenshot 1: errors in console DevTools screenshot 2: 401 in response Works on stable release, 101 in response

I can't track down the cause of clearing Authorization header. Masto.js builds the parameters for websocket connection correctly

[`WebSocketConnectorImpl` calls](https://github.com/neet/masto.js/blob/1ea987575960e1bf937999b0d8e062fe453bd0e4/src/adapters/ws/web-socket-connector.ts#L83) `new WebSocket(...this.props.constructorParameters)` It spreads an array `constructorParameters` that looks like: [URL, [], {headers: {Authorization: ...}}] (open screenshot for details) So WebSocket constructor gets URL, then `[]`, and then...

If the client library doesn't send an access token, i guess WS doesn't work on any server, not only GtS :)

https://github.com/neet/masto.js/commit/932463c03c985edb3a817992f38a42c3f10cc076 already reverted this change (included in release v7.1.0). The latest version of masto.js uses Sec-WebSocket-Protocol again and should work.

Notes on the last commit: * i was particularly surprised by the use of fractional pixels (e.g. `2.5px`) which doesn't make sense (unless a page is scaled in browser) *...

As for https://github.com/chatmail/relay/pull/593/commits/2e8318647a515b94fb42231d9bb6f57ffcc0e071 Sorry, as I see it's the common design for all DeltaChat web pages, I'll return the button border and paddings, but please take a look at what...

> Thanks for looking into this. Truth be told, I'm rather sceptical with this PR. What you call a "mess" (please watch your tone while judging the work of others,...

I glanced at opened PRs and seen no changes correspoding static pages, so I supposed that it's OK to adjust whitespace as there would be no merge conflicts

Another codestyle thing that I've just reverted: single HTML tags. Sometimes they're written as ``, sometimes as ``, sometimes ``. It doesn't change how a page look, yes, but it...