Projects tigase _server server-core Issues #995
XEP-0157: Contact Addresses for XMPP Services (#995)
Closed
wojciech.kapcia@tigase.net opened 6 years ago

This is related to https://compliance.conversations.im/test/abuse-contact/ / XEP-0157: Contact Addresses for XMPP Services but it's actually a sensible idea. It utilizes XEP-0128: Service Discovery Extensions to extend servers disco#info with:

<x xmlns='jabber:x:data' type='result'>
    <field var='FORM_TYPE' type='hidden'>
        <value>http://jabber.org/network/serverinfo</value>
    </field>
    <field var='abuse-addresses'>
        <value>some_address@domain</value>
    </field>
</x>
Andrzej Wójcik (Tigase) commented 6 years ago

I've added support for XEP-0128 in general. Now it is possible to add following entry to the etc/config.tdsl:

'disco-extensions' = {
    'abuse-addresses' = [ 'mailto:support@tigase.net', 'xmpp:admins@tigase.net' ]
}

adds abuse-addresses field with 2 values (as above).

wojciech.kapcia@tigase.net commented 6 years ago

Could you add this to documentation as well?

wojciech.kapcia@tigase.net commented 6 years ago

It works

However it got me thinking. Supposedly we are moving towards "repository-based" configuration, so it would make sense to allow configuring it ad-hoc without the need to restart the instance(s) -- what do you think?

Yes, I thought about applying it to tigase.im instances but instead of simple configuration change this would require going through the whole cycle of creating image…

Andrzej Wójcik (Tigase) commented 6 years ago

We should finish the task with the reconfiguration of the server at runtime. Then moving config file to the shared folder would allow us to easy reconfigure server using ad-hoc commands.

wojciech.kapcia@tigase.net commented 6 years ago

Andrzej Wójcik wrote:

We should finish the task with the reconfiguration of the server at runtime.

Ref: #5489

Then moving config file to the shared folder would allow us to easy reconfigure server using ad-hoc commands.

IMHO we should avoid re-writing configuration file. It was understandable during conversion to TDSL, but changing configuration by re-writing it feels like a bad idea.


After talk with Andrzej we settled on: those configuration options should be part of VHostItem and DefaultVHostItem, created #8791 to track it.

issue 1 of 1
Type
New Feature
Priority
Normal
Assignee
RedmineID
8578
Version
tigase-server-8.1.0
Spent time
17h
Issue Votes (0)
Watchers (0)
Reference
tigase/_server/server-core#995
Please wait...
Page is in error, reload to recover