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

Vendor.js throws error making frontend unusable #232

Closed
RichyHBM opened this issue Sep 16, 2023 · 21 comments
Closed

Vendor.js throws error making frontend unusable #232

RichyHBM opened this issue Sep 16, 2023 · 21 comments
Labels
bug Something isn't working fixed The issue is fixed (in a coming release)
Milestone

Comments

@RichyHBM
Copy link

Version

4.2.x

Details & Steps to reproduce

Having a fully working 4.1.0 setup, upgrade to one of the 4.2.x versions.

I am using Docker with traefik as a reverse proxy and Authelia to provide authentication

Expectation

Latest version works correctly

Error & Logs

vendor.js?id=3f260a606d7ce1f763fd2c9a4ddd334e:1 RangeError: Maximum call stack size exceeded
    at G._t (vendor.js?id=3f260a606d7ce1f763fd2c9a4ddd334e:2:24618)
    at G.t (vendor.js?id=3f260a606d7ce1f763fd2c9a4ddd334e:2:25383)
    at app.js?id=f732c44e609c29cc8eb65402d08d31f8:2:656661
    at v (vendor.js?id=3f260a606d7ce1f763fd2c9a4ddd334e:2:51437)
    at r (vendor.js?id=3f260a606d7ce1f763fd2c9a4ddd334e:2:48069)
    at $t (vendor.js?id=3f260a606d7ce1f763fd2c9a4ddd334e:2:48106)
    at Lt.confirmTransition (vendor.js?id=3f260a606d7ce1f763fd2c9a4ddd334e:2:51827)
    at Lt.transitionTo (vendor.js?id=3f260a606d7ce1f763fd2c9a4ddd334e:2:50154)
    at e.push (vendor.js?id=3f260a606d7ce1f763fd2c9a4ddd334e:2:53210)
    at vendor.js?id=3f260a606d7ce1f763fd2c9a4ddd334e:2:51794
s @ vendor.js?id=3f260a606d7ce1f763fd2c9a4ddd334e:1

The app.js exception is from document.title = e.meta.title = br.app.$options.i18n.t("titles." + e.name),



### Execution environment

Date: Sat, 16 Sep 2023 10:35:18 +0000
userAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/117.0.0.0 Safari/537.36
Version: 4.2.1
Environment: local
Install path: /
Debug: false
Cache driver: file
Log channel: daily
Log level:
DB driver: sqlite
PHP version: 8.1.22
Operating system: Linux
interface: fpm-fcgi
Auth guard: reverse-proxy-guard
Auth proxy logout url:
Auth proxy header for user: HTTP_REMOTE_USER
Auth proxy header for email: HTTP_REMOTE_EMAIL
webauthn user verification: preferred
Trusted proxies: none

### Containerization

- [X] Docker

### Additional information

_No response_
@Bubka
Copy link
Owner

Bubka commented Sep 16, 2023

Hi,

Does ctrl + F5 fix the issue?

@Bubka
Copy link
Owner

Bubka commented Sep 16, 2023

What language do you use?

@RichyHBM
Copy link
Author

Hi, yes I tried doing a full clear cache and tried various browsers and different computers, and even on my phone, all using Brave though.

I don't think I've set any language in the settings so imagine just English

@Bubka Bubka closed this as completed in a2fe6c8 Sep 17, 2023
@Bubka Bubka added bug Something isn't working fixed The issue is fixed (in a coming release) labels Sep 17, 2023
@Bubka Bubka moved this to Done in 2FAuth backlog Sep 17, 2023
@Bubka Bubka added this to the v4.2.2 milestone Sep 17, 2023
@Bubka
Copy link
Owner

Bubka commented Sep 17, 2023

Ok, I just pushed a commit that rolls back 969367e, that should fix the issue.

@RichyHBM
Copy link
Author

Unfortunately this didn't seem to fix it. However I disabled the proxy authentication and reverted to user password and that seems to be fine

@Bubka
Copy link
Owner

Bubka commented Sep 18, 2023

That should have nothing to do with the proxy auth 🤨
Does the issue recur if you re-enable the proxy auth?

@Bubka Bubka reopened this Sep 18, 2023
@github-project-automation github-project-automation bot moved this from Done to In Progress in 2FAuth backlog Sep 18, 2023
@RichyHBM
Copy link
Author

It does, yes

@RichyHBM
Copy link
Author

Has anything changed with proxy with since 4.1? As it worked fine there but seems like it's not logging me in properly in 4.2

@Bubka
Copy link
Owner

Bubka commented Sep 18, 2023

Not directly.
I was able to reproduce the issue, this is indeed triggered when the proxy authentication is On, I think it is a side effect of another change. I'm working on it.

@Bubka
Copy link
Owner

Bubka commented Sep 19, 2023

I just pushed a new fix on the dev branch (2fauth/2fauth:dev). Can you test it?

@Bubka Bubka modified the milestones: v4.2.2, v4.2.3 Sep 19, 2023
@RichyHBM
Copy link
Author

Looks like I'm getting the same issue on dev

@Bubka
Copy link
Owner

Bubka commented Sep 20, 2023

You still have a js exception of type RangeError: Maximum call stack size exceeded !?

@RichyHBM
Copy link
Author

RichyHBM commented Sep 20, 2023

I did, yea. Saying that I didn't do a full cache clear, let me try that

@RichyHBM
Copy link
Author

Hmm yes, tried it in incognito and same issue

@Bubka
Copy link
Owner

Bubka commented Sep 20, 2023

How do you do the full cache clear?

Because I had a really weird issue on my dev env these days and despite a disabled cache and the use of ctrl+F5 I couldn't have my front-end working correctly. The only way I found to fix it was to click the 🔒 icon in the address bar (I'm using Firefox) and use the Clear cookies and site data feature.

@RichyHBM
Copy link
Author

I chrome based browsers, I open the dev tools, then can hold the refresh button and then there is a drop-down with hard refresh+empty cache. But I've also tried in incognito

@peterbuga
Copy link

can confirm, the problem persists. same setup, traefik w/ header auth Remote-User

@Bubka
Copy link
Owner

Bubka commented Sep 24, 2023

I just pushed a new fix on the dev branch (2fauth/2fauth:dev). Can you test it?

Sorry for the late response. I just pushed a missing commit on Dev which rebuild js assets, this should fix the issue this time. Can you confirm please.

@RichyHBM
Copy link
Author

That looks to have fixed it

@RichyHBM
Copy link
Author

Not sure how big of a trouble it could be but may be worth moving the asset rebuild as a part of the docker build, that way you don't have to remember to do it manually

@Bubka Bubka moved this from In Progress to Done in 2FAuth backlog Sep 26, 2023
@Bubka
Copy link
Owner

Bubka commented Sep 26, 2023

Not sure how big of a trouble it could be but may be worth moving the asset rebuild as a part of the docker build, that way you don't have to remember to do it manually

You're right. It's in my (way too long) todo list by the way.
For now v4.2.3 has just been release, with the bug fix 😅

@Bubka Bubka closed this as completed Sep 26, 2023
@Bubka Bubka moved this from Done to Released in 2FAuth backlog Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed The issue is fixed (in a coming release)
Projects
Status: Released
Development

No branches or pull requests

3 participants