Raised This Month: $32 Target: $400
 8% 

Add Admin IP Fix


Post New Thread Reply   
 
Thread Tools Display Modes
Author Message
hleV
Veteran Member
Join Date: Mar 2007
Location: Lithuania
Old 08-18-2009 , 10:23   Add Admin IP Fix
Reply With Quote #1

Have you ever tried to add admin by IP address by console command "amx_addadmin"? Well, it doesn't work. It works like this be default:

E.g. if you're performing this command (player with that IP must be in server):
Code:
amx_addadmin 123.45.67.89 abcdefghijklmnopqrstu "" ip
It writes this (no matter SQL or not):
Quote:
"123.45.67.89:27005" "" "abcdefghijklmnopqrstu" "de"
Yes, it adds client's port that is unnecessary and fucks everything up. Here's the fix for it.

admin.sma line 254:
Code:
                        get_user_ip(player, auth, 32)

Code:
                        get_user_ip(player, auth, 32, 1)
Attached Files
File Type: sma Get Plugin or Get Source (admin.sma - 1138 views - 19.9 KB)
File Type: sma Get Plugin or Get Source (admin_sql.sma - 1039 views - 19.9 KB)
__________________
hleV is offline
crazyeffect
Veteran Member
Join Date: Jul 2008
Location: Belgium
Old 08-18-2009 , 10:32   Re: Add Admin IP Fix
Reply With Quote #2

Wow!
__________________
crazyeffect is offline
Send a message via MSN to crazyeffect
xPaw
Retired AMX Mod X Moderator
Join Date: Jul 2008
Old 08-18-2009 , 10:43   Re: Add Admin IP Fix
Reply With Quote #3

steamids ftw
__________________
xPaw is offline
ot_207
Veteran Member
Join Date: Jan 2008
Location: Romania The Love Country
Old 08-18-2009 , 11:34   Re: Add Admin IP Fix
Reply With Quote #4

I think that you should have better filed a bug report .
__________________
My approved plug-ins | Good for newbies! | Problems?

Back, will come around when I have time.
ot_207 is offline
Old 08-18-2009, 14:15
hleV
This message has been deleted by Emp`. Reason: Double Post
hleV
Veteran Member
Join Date: Mar 2007
Location: Lithuania
Old 08-18-2009 , 14:17   Re: Add Admin IP Fix
Reply With Quote #5

Quote:
Originally Posted by xPaw View Post
steamids ftw
Spam FTW.
Quote:
Originally Posted by ot_207 View Post
I think that you should have better filed a bug report .
I guess so. I'm used to fix the bugs myself though.

Well, I hope someone will do that since I'm not planning to.
__________________
hleV is offline
Styles
Veteran Member
Join Date: Jul 2004
Location: California
Old 08-19-2009 , 23:56   Re: Add Admin IP Fix
Reply With Quote #6

rofl I don't think they want to add another version / fix. I mean CelliTri or w/e is known to have that string fucntion to be busted. Hawk even has a fix, he gave me it and I don't think they are updating with that.
Styles is offline
Send a message via AIM to Styles
Hawk552
AMX Mod X Moderator
Join Date: Aug 2005
Old 08-20-2009 , 03:01   Re: Add Admin IP Fix
Reply With Quote #7

If you post it as a bug report I'll commit it but I dunno whether a version will actually get released.
__________________
Hawk552 is offline
Send a message via AIM to Hawk552
ot_207
Veteran Member
Join Date: Jan 2008
Location: Romania The Love Country
Old 08-20-2009 , 03:30   Re: Add Admin IP Fix
Reply With Quote #8

Quote:
Originally Posted by Hawk552 View Post
If you post it as a bug report I'll commit it but I dunno whether a version will actually get released.
In the next version of amxx probably.
__________________
My approved plug-ins | Good for newbies! | Problems?

Back, will come around when I have time.
ot_207 is offline
Reply



Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT -4. The time now is 20:33.


Powered by vBulletin®
Copyright ©2000 - 2024, vBulletin Solutions, Inc.
Theme made by Freecode