summaryrefslogtreecommitdiffstats
path: root/src/account-server
Commit message (Collapse)AuthorAgeFilesLines
...
* General cleanups and code style fixesThorbjørn Lindeijer2011-03-205-106/+88
| | | | | | | A bunch of variables that were continuously requested from the configuration are now members of the AccountHandler. Reviewed-by: Freeyorp
* Improved @ban commandPhilipp Sehmisch2011-03-182-2/+5
| | | | | | | | | | | When banning a character, the game master now sets a time unit (m, h, d, w or y for minutes, hours, days, weeks or years) after the duration. Ban durations longer than 2^16 minutes are now possible. The banned character is now kicked automatically and the banning character receives a feedback chat message. Reviewed-by: Thorbjorn
* Micro-optimizations related to std::stringThorbjørn Lindeijer2011-03-173-11/+13
| | | | | | | | * Rely on the fact that a std::string is empty by default * Use std::string::empty() rather than comparing to "" * Construct with std::string() rather than from "" Reviewed-by: Bertram
* Moved some file-global variables into the AccountHandlerThorbjørn Lindeijer2011-03-151-42/+40
| | | | | | | Nicer as members where there is clear ownership, initialization and scope. Reviewed-by: Freeyorp
* Introduced an AttributeValue class for convenience and readabilityThorbjørn Lindeijer2011-03-153-29/+43
| | | | | | | | Easier to understand than a std::pair with its 'first' and 'second' members, and it also provides an implicit constructor so that AttributeValue is implicitly constructed from a double. Reviewed-by: Freeyorp
* Made the "No game server for map" error message report the map IDPhilipp Sehmisch2011-03-141-1/+1
| | | | (trivial patch)
* Removed a bunch of silly documentationThorbjørn Lindeijer2011-03-123-17/+2
| | | | | | A C++ developer should be able to recognize a constructor and a destructor by just looking at it, so let's stop writing down the obvious. :)
* Prevented a potential crash in storage::getAllWorldStateVars()Yohann Ferreira2011-03-091-2/+10
| | | | | | Minor other fixes. Trivial.
* Implemented persistent world and map variablesPhilipp Sehmisch2011-03-043-3/+98
| | | | | | | | | | | | | | | | | | | | | The gameserver now receive a copy of all world state variables when they are accepted by the accountserver and receive a copy of all map state variables of a map when they register it successfully. Implemented LUA script bindings for getting and setting these variables. When such a variable is set, the accountserver is notified about this change. Changes to world state variables are then propagated to all gameservers by the accountserver. Be aware that when a gameserver is updating a map, there is no check if it is actually responsible for said map. But I consider this not a security flaw, because authenticated game servers are considered to be trustworthy in a lot of other situations, too. Also renamed "quest" to "character variable" in the sourcecode. Reviewed-by: Bertram
* Remove obsolete reset of SIGSEGV handlingStefan Dombrowski2011-02-211-3/+0
| | | | Reviewed-by: Thorbjorn
* Removing dublicate deinitialize of enetStefan Dombrowski2011-02-191-3/+0
| | | | | | Both servers deinitialize enet now from deinitializeServer(). Reviewed-by: Bertram, Jaxad0127
* Removing unused variable configPathChangedStefan Dombrowski2011-02-161-3/+0
|
* Fix the GP base and mod values for newly created characters.Yohann Ferreira2011-02-111-0/+11
| | | | Reviewed-by: Freeyorp.
* Add an heartbeat time flag in the statistics file.Yohann Ferreira2011-01-261-2/+13
| | | | | | | | | | | | | | | | | | This way, any kind of external service can know about the running state of each servers, even if only the account server is updating the statistics because: 1. When the account server is running, each <gameserver> tag is a running game server. Otherwise, it's not listed. 2. Whenever the account server stops running, the stat file isn't updated anymore and so the <heartbeat> tag isn't. 3. Game servers without any contact with an account server will try to reconnect to the account server and kick out every players so they aren't considered as running in that case. Reviewed-by: Crush. Resolves: Mana-mantis #270.
* Made the servers report their protocol version.Yohann Ferreira2011-01-112-3/+6
| | | | | | | | | | | | | As the protocol version isn't something the maintainers should set, I removed the net_clientVersion irrelevant option, replaced by the corresponding define PROTOCOL_VERSION in the manaserv_protocol.h file. A corresponding change in the client will be made. N.B.: Now, the Enet version is also reported in the log. Reviewed-by: Jaxad.
* Removed the superfluous point struct.Yohann Ferreira2011-01-093-4/+5
| | | | | | | | It was too close from the Position class and it leads to making the server handle one or another type through the code. Still bugged me many times while making changes. Reviewed-by: Jaxad.
* Namespaced the manaserv protocol to ease the porting of the enums.Yohann Ferreira2011-01-032-0/+4
| | | | Trivial.
* protocol.h -> manaserv_protocol.h to avoid conflicts with mysql files.Yohann Ferreira2010-12-292-2/+2
| | | | Resolves: Mana-Mantis #278.
* Made the server handle properly the characters slots.Yohann Ferreira2010-12-297-68/+212
| | | | | | | | | I turned the vector storing character data into a map, keeping the character's slot. Fixed a memleak along the way. Reviewed-by: Crush.
* Transaction logging of various command, new transaction codes and style fix.Philipp Sehmisch2010-12-271-1/+1
| | | | | | | | | | | | | | The @mute, @goto, @attribute and @announce commands now logs transactions to the database. Added new transaction codes to database. I've preliminarily added a lot of codes for not yet supported commands so that we needn't do a database update with every single commit which implements one. Also using the equivalent "say" helper function in place of the "GameState::sayTo" method in the command handler. Reviewed-by: Kage
* Fixed a compiler warning.Philipp Sehmisch2010-12-171-1/+1
|
* Added last missing try catch implementation in storage.cppYohann Ferreira2010-12-161-103/+175
| | | | | | Resolves: Mana-mantis #269. Reviewed-by: Jaxad0127.
* Use bound values for Storage::flush() since it leaked sensitive info.Yohann Ferreira2010-12-161-7/+19
| | | | | | (Readable in the log with debug level.) Reviewed-by: Jaxad0127.
* Fix existing binding use in storage.cpp.Yohann Ferreira2010-12-161-55/+98
| | | | Reviewed-by: Jaxad0127.
* Deharcoded the host and port options for each servers.Yohann Ferreira2010-12-162-9/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changes: ============================================================ For the account server: ------------------------------------------------------------ Old -> New ------------------------------------------------------------ net_accountServerAddress, net_listenHost -> net_accountHost net_accountServerPort -> net_accountListenToClientPort net_accountServerPort +1 -> net_accountListenToGamePort For the game server: ------------------------------------------------------------ Old -> New ------------------------------------------------------------ net_gameServerAddress -> net_gameHost net_gameServerPort -> net_gameListenToClientPort For the chat server: ------------------------------------------------------------ Old -> New ------------------------------------------------------------ -> net_chatHost net_accountServerPort + 2 -> net_chatListenToClientPort Special fallback feature, as requested by Freeyorp: When the net_accountListenToClientPort (default to 9601) is set, the 3 others ports will automatically offset from it, if they're not set, following this rule: net_accountListenToGamePort = net_accountListenToClientPort + 1 net_chatListenToClientPort = net_accountListenToClientPort + 2 net_gameListenToClientPort = net_accountListenToClientPort + 3 Resolves: Mana-Mantis #216. Reviewed-by: Jaxad0127.
* Standardized the current errors thrown.Yohann Ferreira2010-12-161-110/+85
| | | | Reviewed-by: Jaxad0127.
* Dummy commit since it seems my private repo didn't want to updateYohann Ferreira2010-12-141-0/+1
| | | | the public one.
* Remove an outdated comment talking about subversion tagging.Yohann Ferreira2010-12-141-3/+1
|
* Removed the code preventing from being able to sync items with id < 500.Yohann Ferreira2010-12-141-4/+1
| | | | | | Instead, let's use the actual rule: Id > 0. Trivial fix.
* Second round of small random cleanups on storage.Yohann Ferreira2010-12-142-63/+59
| | | | Trivial also.
* Small random hacking cleanups.Yohann Ferreira2010-12-141-163/+189
| | | | Trivial.
* Moved storage doc from .cpp to .h and updated it.Yohann Ferreira2010-12-132-248/+311
| | | | | | Since it's what is done for the other files in the project. Trivial fix.
* Fix many compiler warnings.Freeyorp2010-12-081-1/+1
| | | | Reviewed-by: Bertram
* Made the server return the number of slots in the login/register responses.Yohann Ferreira2010-11-181-3/+14
| | | | Reviewed-by: CodyMartin.
* Upgrade database version to 12 as it should have been from the latest update.Yohann Ferreira2010-11-151-1/+1
| | | | Trivial fix.
* Renamed .hpp files into .h for consistency.Yohann Ferreira2010-11-1413-64/+64
| | | | | | Also added an header to the autoattack.{h,cpp} files. Big but trivial fix.
* Fix code indentation in the sql related files.Yohann Ferreira2010-11-101-29/+37
| | | | Trivial fix.
* Fixed unregistering on the server-side.Yohann Ferreira2010-11-092-9/+20
| | | | | | | | | | | | | | | | When registering or logging, The client is hashing the password for sending it safely. And the server is hashing it also to store it the same way. Hence, the password ends hashed twice, which is correct because the server can't trust the client anyway. At unregister attempt, the server wasn't hashing the password before comparing it. Also while on it, I made the corresponding SQL query use the try catch method and only delete the account in memory when it's also done on the Db. Reviewed-by: thorbjorn, Freeyorp.
* Renamed write{Byte,Short,Long} to writeInt{8,16,32}Thorbjørn Lindeijer2010-11-032-141/+141
| | | | | Mainly for consistency with the client, and the general consensus was that these numbered versions were clearer.
* Have the PerformTransaction class automatically handle nestingThorbjørn Lindeijer2010-10-302-70/+17
| | | | | | No need to fiddle around with "startTransaction" booleans now that the helper class is a little more intelligent. When the database is already performing a transaction, no new one will be started.
* Introduced a helper class for performing transactionsThorbjørn Lindeijer2010-10-302-14/+12
| | | | | | Avoids having to remember to call rollbackTransaction and makes transactions exception-safe (since the destructor of PerformTransaction will be called when an exception is thrown).
* Use a transaction when handling a GAMSG_PLAYER_SYNC messageThorbjørn Lindeijer2010-10-302-11/+21
| | | | | | | | | | | This message can contain a lot of small database updates, which at least on my system are way more efficient when performed in a transaction (now it takes no more than 1 second vs. about 14 seconds before). Not saying this is normal, my guess is that it's due to using full partition encryption. I've also prevented the thing from entering an infinite loop in the case of a wrong message, and corrected some variable names.
* Send a client data URL in addition to the update hostThorbjørn Lindeijer2010-10-261-0/+7
| | | | | | | | | | This is for developing/testing an experimental new resource manager that downloads only the files it needs on demand. Currently only used by the Mana Mobile project. Regular Mana client ignores the setting. Reviewed-by: Jared Adams
* Fixed the append behaviour of logger::setLogFile().Yohann Ferreira2010-10-201-1/+1
| | | | | | | | | Also made both servers appends on the main log at start if they aren't empty. This permits not to lose the previous logs when quicky restarting the servers. Trivial fix.
* Synchronise defaults between configuration file and source.Freeyorp2010-10-171-1/+1
| | | | Reviewed-by: Bertram
* Add log file rotation support based on ExceptionFault's work.Yohann Ferreira2010-09-291-0/+10
| | | | | | | | | This patch adds options to enable log rotations base on files size and or change of date. Note: Zip support will be added in a second commit. Reviewed-by: CodyMartin, Thorbjorn.
* Added options to disable the server log to standard output to the minimum.Yohann Ferreira2010-09-221-10/+13
| | | | | | | | Also made some small cleanups. Resolves: Mana-Mantis #241 Reviewed-by: Jaxad0127.
* Centralized the servers' exit values.Yohann Ferreira2010-09-222-18/+19
| | | | | | | | Also made random code format clean-ups. Resolves: Mana-Mantis #217 Reviewed-by: Jaxad0127.
* Reordered the manaserv.xml.example file and made related code cleanups.Yohann Ferreira2010-09-152-5/+6
| | | | Reviewed-by: Thorbjorn.
* Centralized stringToBool conversion.Yohann Ferreira2010-09-081-19/+27
| | | | | | | Also moved the trim() function into the utils namespace where it belongs more, and made some random code cleanups. Reviewed-by: Thorbjorn.