View Issue Details

IDProjectCategoryView StatusLast Update
0001916unrealircdpublic2006-04-27 19:12
ReporterStealth Assigned To 
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionno change required 
Summary0001916: Duplicate modes
DescriptionI think Unreal should treat modes from services as it would from users.

Like if someone has op on a channel, and the services send a +o, the op from services also shows.

Another thing that is somewhat annoying, is when services deop someone. If the person is just +o on a chan, and the services send -qaohv, all the clients see all the modes.

So basically what I am saying, is send the modes by services to the clients the same way as it would send a normal mode.
3rd party modules

Activities

Praetorian_

2004-07-04 03:25

reporter   ~0006857

You mean parse and actually see if the modes were really set, and not just do whatever it is sent?

I believe the line:
'Servers are deemed to be sane, therefore they know what they are doing'.

White_Magic

2004-07-04 19:30

reporter   ~0006862

doesnt service store all the mode on flags? - which means it shouldnt need to do it to eveyr mode as " services know what they are doing " if they store the modes as flags it should only be removing the flags present and not ones that are not there??

question... How can u remove something you dont have? O_o desync? o_O

Stealth

2004-07-05 18:52

reporter   ~0006873

When the services send a mode change, whatever is sent from the services is sent to the clients.

Example:
* Nick (user@host) has joined #channel
* irc.mynet.com sets mode: +o Nick
* ChanServ sets mode: -qaoh Nick Nick Nick Nick

Should be:
* Nick (user@host) has joined #channel
* irc.mynet.com sets mode: +o Nick
* ChanServ sets mode: -o Nick

Unreal would see that the services are sending mode changes that cannot be done, and only send the proper changes to the clients.

Rocko

2004-07-06 12:46

reporter   ~0006881

Bugz, if this disturbs you, then send an EMAIL to the services coder !
THAT is NOT an IRCd issue, that should be changed.

vonitsanet

2004-07-06 12:50

reporter   ~0006882

This is not happening on IrcServices 5.0.34

Rocko

2004-07-06 12:53

reporter   ~0006885

And neither on kickservices.

codemastr

2004-07-06 12:55

reporter   ~0006887

The services are functioning *exactly* as they should. In my mind, it's neither a services nor an IRCd bug, both are functioning exactly as they were designed.

vonitsanet

2004-07-06 12:58

reporter   ~0006888

I think that unrealircd generally ignores -h if you don't have +h and -o if you don't have +o ETC but a U:Lined server can do anything (RAW)..

codemastr

2004-07-06 14:01

reporter   ~0006895

Yes, that's correct. And that is how it is designed.

Zell

2004-07-06 22:01

reporter   ~0006909

I agree with codemastr, its all coded properly.... that a U:Lined server has permisson to perform any command it normally wouldn't be able to do; i.e. setting excess (and duplicate) +o modes, etc. If that is a big problem that all the modes are unset when they weren't set in the first place, post on your services' bug board for a feature request to make services only unset the modes that are active. (but really, the services is only being protective, as they should be.)

syzop

2006-04-27 19:12

administrator   ~0011635

as mentioned a few times (in the future and here in this exact bugreport), this is correct behavior. this will stay as-is.

Issue History

Date Modified Username Field Change
2004-07-02 16:17 Stealth New Issue
2004-07-04 03:25 Praetorian_ Note Added: 0006857
2004-07-04 19:30 White_Magic Note Added: 0006862
2004-07-05 18:52 Stealth Note Added: 0006873
2004-07-06 12:46 Rocko Note Added: 0006881
2004-07-06 12:50 vonitsanet Note Added: 0006882
2004-07-06 12:53 Rocko Note Added: 0006885
2004-07-06 12:55 codemastr Note Added: 0006887
2004-07-06 12:58 vonitsanet Note Added: 0006888
2004-07-06 14:01 codemastr Note Added: 0006895
2004-07-06 22:01 Zell Note Added: 0006909
2006-04-27 19:12 syzop Status new => closed
2006-04-27 19:12 syzop Note Added: 0011635
2006-04-27 19:12 syzop Resolution open => no change required