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

telescope_entries table queries are shown in Query Watcher #1126

Closed
ismaail opened this issue Sep 14, 2021 · 7 comments · Fixed by #1127
Closed

telescope_entries table queries are shown in Query Watcher #1126

ismaail opened this issue Sep 14, 2021 · 7 comments · Fixed by #1127

Comments

@ismaail
Copy link

ismaail commented Sep 14, 2021

  • Telescope Version: v4.6.3
  • Laravel Version: v8.61.0
  • PHP Version: 8.0.10
  • Database Driver & Version: MySQL 8

Description:

telescope_entries table queries are shown in Query Watcher.

telescope.issue.mp4

Steps To Reproduce:

  1. Upgrade Laravel Telescope to version 4.6.3
  2. Have TELESCOPE_DOMAIN in the .env file with empty value or remove it.
@RTippin
Copy link

RTippin commented Sep 15, 2021

I can confirm having this behavior as well after updating to v4.6.3. Even my request watcher was showing all telescope POST to the internal telescope-api.

@metalogico
Copy link

Yes, same problem for me. I just left a comment on the commit which is likely to cause the issue.

@SaaberDev
Copy link

facing same issue .... 😫😫

@ismaail
Copy link
Author

ismaail commented Sep 16, 2021

@SaaberDev workaround: add TELESCOPE_DOMAIN to .env with the app domain as its value.

@SaaberDev
Copy link

SaaberDev commented Sep 16, 2021

@ismaail

Not working ....

@ismaail
Copy link
Author

ismaail commented Sep 16, 2021

@SaaberDev try without https://

@SaaberDev
Copy link

@taylorotwell .... Thanks boss .. Got the new update just now and fixed ❤❤

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 a pull request may close this issue.

4 participants