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

Integration hooks not detected. #3682

Open
tinoest opened this issue Dec 22, 2022 · 4 comments
Open

Integration hooks not detected. #3682

tinoest opened this issue Dec 22, 2022 · 4 comments
Labels
Milestone

Comments

@tinoest
Copy link
Contributor

tinoest commented Dec 22, 2022

In the latest development version of Elkarte the integration hooks for TinyPortal are not detected.

I think it is due to the file only being read in for the first hook. As adding an else to the if/elseif block results in it falling through to there. I would expect the preg_match to catch it.

$fp = fopen($file['dir'] . '/' . $file['name'], 'rb');

@Spuds Spuds added the bug label Dec 22, 2022
@Spuds Spuds added this to the 2.0 milestone Dec 22, 2022
@Spuds
Copy link
Contributor

Spuds commented Dec 22, 2022

Are you seeing that in the maintenance -> integration hooks function ?

@tinoest
Copy link
Contributor Author

tinoest commented Dec 22, 2022

Sorry it was this path.

index.php?action=admin;area=maintain;sa=hooks;

Screenshot 2022-12-22 at 20 29 40

@emanuele45
Copy link
Contributor

Hey, long time no see... :-\
I was wondering: does this page really help in any way?
I mean, IIRC the management page was added just as a way to reassure some people that hooks were not evil when "some people" at the times of SMF 2.1-alpha were raising all sort of conspiracy theories about presence of hooks and possibilities of being hacked and all sort of stuff.
I always found it rather tricky to maintain and actually a potential problem in exposing internal functioning of hooks that for the average admin would have been pretty tricky if tinkered with. 🤷‍♂️
As usual, I start suggesting to kill stuff... 😆

@Spuds
Copy link
Contributor

Spuds commented Dec 4, 2023

I'm never opposed to removing stuff, especially when it does not provide a clear benefit. Oh hooks are are evil, hard editing FTW

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants