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

ntop.conf not generated/doesn't work #71

Open
hikigaya5 opened this issue Sep 7, 2024 · 7 comments
Open

ntop.conf not generated/doesn't work #71

hikigaya5 opened this issue Sep 7, 2024 · 7 comments

Comments

@hikigaya5
Copy link

I was searching for the ntop.conf file to change the color scheme but I didn't find it anywhere. I created my own ntop.conf file and located it in the same folder as the executable as I saw in another issue but this doesn't seem to work when I try to change colors.
Also the link in the ntop.conf example file of the readme doesn't direct you to the page of the possible colors, it redirects you to other thing.

@Demitrix2k
Copy link

I tried to do the same using the template and modifying it, placing it alongside ntop.exe in the folder structure for

AppData\Local\Microsoft\WinGet\Packages\gsass1.NTop_Microsoft.Winget.Source_8wekyb3d8bbwe

tried if i could set Env:NTOP_CONFIG_HOME= no such luck either.

I see people back to 2021 with questions about this not answered, would appreciate one and possibly updating main page regarding this.

@gsass1
Copy link
Owner

gsass1 commented Oct 22, 2024

@hikigaya5 @Demitrix2k

It seems originally I only implemented config file reading, not config file generation

https://github.com/gsass1/NTop/blob/master/ntop.c#L170

Have you tried copying ntop.conf from this repo to the same dir as ntop.exe?

@hikigaya5
Copy link
Author

Yes I tried. Also where are the possible colors? Because the link in the generated ntop.conf file redirects you to a CHAR_INFO structure that says it has only 3 possible colors (red, green and blue) and it doesn't seem quite clear to me.

@gsass1
Copy link
Owner

gsass1 commented Oct 22, 2024

@hikigaya5 The way it's currently implemented is not very user friendly. You have to currently manually create the RGB value, I think there's a tool for this but I'm not sure

@Demitrix2k
Copy link

@hikigaya5 @Demitrix2k

It seems originally I only implemented config file reading, not config file generation

https://github.com/gsass1/NTop/blob/master/ntop.c#L170

Have you tried copying ntop.conf from this repo to the same dir as ntop.exe?

I have, i have also edited the config and adjusted the color values to test for visual changes just to see if anything would result of it, such as setting every color option to '0x2' or '0x40' as they already were present in the config.

@gsass1
Copy link
Owner

gsass1 commented Oct 26, 2024

In the newest commit the config file is read relative to the executable file (the intended behaviour, before it was working directory).

I tested this and it should work, let me know!

@Demitrix2k
Copy link

Can confirm, now works with config file within the same folder as the executable.

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

No branches or pull requests

3 participants