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

Ability to set RemoteNickFormat for Matterbridge #4279

Open
markuman opened this issue Oct 4, 2020 · 6 comments
Open

Ability to set RemoteNickFormat for Matterbridge #4279

markuman opened this issue Oct 4, 2020 · 6 comments
Labels
1. to develop enhancement feature: integration 📦 Integration with 3rd party (chat) service
Milestone

Comments

@markuman
Copy link

markuman commented Oct 4, 2020

Is your feature request related to a problem? Please describe.

No

Describe the solution you'd like

When using this for single user setup, it may annoy others on the target chat system when they see always

[nctalk] <markuman> my message
[nctalk] <markuman> bla blabla

So it would be nice to disable/overwrite it.

Describe alternatives you've considered

Dirty patch it like this

sed -i 's/\[{PROTOCOL}\] <{NICK}> //g' lib/MatterbridgeManager.php

Additional context

Screenshot_2020-10-04 Glowing Bear #ansible-aws Ansible + AWS https github com ansible community wiki aws

@nickvergessen
Copy link
Member

Well you can also adjust your matterbridge config file in tmp

@markuman
Copy link
Author

markuman commented Oct 4, 2020

@nickvergessen
/tmp is on most systems not persistent I think.
And my setup is using docker.
So I think I can automate it for my needs. But it would be nice to overwrite this settings/variables.

@nickvergessen nickvergessen added 1. to develop feature: integration 📦 Integration with 3rd party (chat) service and removed 0. Needs triage labels Oct 4, 2020
@nickvergessen
Copy link
Member

Just ment its the better hack then adjusting the Talk source code

@nickvergessen nickvergessen added this to the 💔 Backlog milestone Oct 4, 2020
@markuman
Copy link
Author

markuman commented Oct 4, 2020

@nickvergessen

Just ment its the better hack then adjusting the Talk source code

True. But every click in the settings seems to overwrite the .toml file.
I need to mount it as :ro in the container or do chmod 444 on that file.

@markuman
Copy link
Author

markuman commented Oct 4, 2020

Uh, I figured out that the config is not persistent at all.

After restart

cat /tmp/bridge-8j5ditpi.log 
time="2020-10-04T17:44:45Z" level=info msg="Running version 1.18.3 8b26e42a" prefix=main
time="2020-10-04T17:44:45Z" level=fatal msg="Failed to read configuration file: &os.PathError{Op:"open", Path:"/tmp/bridge-8j5ditpi.toml", Err:0x2}" prefix=config

But in the right sidebar, the information are still present. So I need to click "save" again and hit several times on "enabled" until it gets the status "running".

@leArthurDent
Copy link

I'd just like to say that I'd also like this feature very much. Sadly, the toml files are not persistent and I haven't found a way to tweak the matterbidge config, except for adjusting MatterbridgeManager.php.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop enhancement feature: integration 📦 Integration with 3rd party (chat) service
Projects
None yet
Development

No branches or pull requests

3 participants