search

EVE-SECURE / temars-eve-api

Automatically exported from code.google.com/p/temars-eve-api
0 stars 0 forks source link

TEA_TS doesnt remove old ts users #60

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
TEA Version?
1.2.0
SMF Version?
2.0 rc5

What steps will reproduce the problem?
1. Register a user on ts with UID1, then disconnect.
2. Connect the same user to TS with UID2
3. Goto TS page and click change profile.
4. As admin, remove the users groups on SMF, to simulate a failed api key or 
user leaving corp/alliance.
5. Run TEA_TS_Cron_Access.php
6. Connect to TS with UID1

What is the expected output? What do you see instead?

It should remove the roles from UID1, then add them to UID2, instead, it just 
adds the roles to UID2 and leaves UID1 with the roles it had, causing a problem 
when the API check fails as it will remove the roles from UID2, but it has 
forgotten about UID1, leaving it with all the roles it had.

Please provide any additional information below.

Seems like TEA_TS_Cron_Access.php isnt checking the TS groups against its list 
of allowed people and doesnt remove UID's that TEA isnt expecting.

It does change the UID linked with that account, which is expected.

Original issue reported on code.google.com by DTHI.Dem...@gmail.com on 15 Jul 2011 at 4:08

GoogleCodeExporter commented 9 years ago 
ah yes I did notice using online method doesn't remove old when adding new
unlike unique key method which does

although I didn't think of it in that was I was just like oh well some old info 
in database
not oh old info that people can use to get bk on when they shouldn't :p

ill fix it tomorrow as im about to go bed

Original comment by tema...@googlemail.com on 15 Jul 2011 at 4:16

GoogleCodeExporter commented 9 years ago 
Thanks, i tried to goto bed and failed :P.

Original comment by DTHI.Dem...@gmail.com on 15 Jul 2011 at 4:52

GoogleCodeExporter commented 9 years ago

Original comment by tema...@googlemail.com on 15 Jul 2011 at 7:34

GoogleCodeExporter commented 9 years ago

Original comment by tema...@googlemail.com on 14 Aug 2011 at 3:07