| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
|
|
|
| |
As seems login forgot to close a parenthesis in the manual recovery
messages, and also appears like I am hunting for errors in tcoin :)
I have just changed the instances of the manual recovery messages from
"login@tilde.town (town-only) or login@tilde.team (internet wide [...]"
to:
"login@tilde.town (town-only) or login@tilde.team (internet wide) [...]"
|
| |
|
|
|
| |
I've tried to add try-catch blocks in those cases where fork()s fail.
It doesn't quite work well though.
|
| |
|
|
| |
I can't believe I made this mistake, but I doublecounted minercoins, so now that doesn't happen.
|
| |
|
|
| |
Adding jmjl's 'ju coin' (which is a copy of tildebot) to the tcoin and pcoin balance
|
| |
|
|
| |
Due to krowbar not being here and krowbar's tildebot not rejoining, shifting tildescores file location to jmjl's tildescores location. jmjl is running krobwar's tildebot with the name 'ju' in tilde.town's internal irc's #bots channel
|
| |
|
|
|
|
|
|
| |
old "~username" format
Made this change so that new "!minercoin ask" awarded minercoins will still show up in tcoin without losing old !minercoins awarded.
In other words, the inflation will continue until morale improves.
|
| |
|
|
| |
Thanks to ~jmjl, I was able to find that pcoin_keys didn't work. To fix it, tcoin uses sneaky magic but with stdin isntead of an argument, and a new library imported called popen2, that lets me send both stdin and receive stdout, to successfully get "ls <path to the pcoin_keys directory>" to work (it execl's another instance of the tcoin comand so that it runs under setuid instead of as the original user who ran it.
|
| |
|
|
| |
Simple logical mistake in checking for the TILDEGURU flag has been corrected in tcoin.cpp
|
| |
|
|
| |
/usr/bin/whoami doesn't give the euid's username, but the uid's on FreeBSD, so did the same thing I did for tilde.institute, but for tilde.guru too and introduced a new flag for it, TILDEGURU.
|
| |
|
|
|
|
|
|
|
|
| |
single file
This is preparing tcoin/pcoin for the automatic registration of program accounts.
Eventually, I want program accounts to be tradeable by there being a way to regenerate a new key.
So, send the current key to whomever you want the program account to be used by, then the new owner regenerates the key
to complete the transfer. The old owner will no longer be able to use the old key to operate the new account. The key is
long and the key generation random enough for key collisions to be virtually impossible.
|
| |
|
|
|
|
| |
Improved the way 5 arguments were handled, where the intention was to
send a message along with the amount and receiver, with a new error code
and a slightly modified error message.
|
| |
|
|
|
| |
Fixed a case of too many arguments in 'tcoin send' that wasn't handled.
Thank you for finding this ~julian.
|
| |
|
|
|
|
|
|
| |
There were many more places where I had hardcoded values, but
should have used strlen() or sizeof(). I have gone through
tcoin.cpp and pcoin.cpp, found those, and fixed them.
Hopefully, this is all of them.
|
| |
|
|
|
|
|
|
| |
In user_has_initialised(..), the path lengths were hardcoded.
This has now been changed to being calculated via sizeof().
In get_file_value and add_file_value, strlen() was used. This has
also been replaced with sizeof().
|
| |
|
|
|
| |
Added comment to tcoin.cpp to send the actual error message to
stderr when running exit_program(err_no).
|
| |
|
|
|
| |
Refactored the send function in tcoin and pcoin, and in the process,
fixed a bug in pcoin's silentsend function.
|
| |
|
| |
The comment referred to using || instead of &&, despite the fact that the code now uses &&.
|
| |
|
|
| |
On OpenBSD, the equivalent of %_d is %e. This is fixed for OpenBSD/tilde.institute type systems now.
|
| |
|
|
|
|
| |
I was planning to make a change that I did not adequately reverse, so I have reversed it fully now.
The change I was planning to make was to remove references to login@tilde.town on systems that were
not tilde.town, but I decided against that change eventually (but didn't reverse it fully).
|
| |
|
|
|
|
|
| |
1. Custom host name now possible in tcoin --help
2. Changes to ntcoin and ntcoin_openbsd_tildeinstitute to fix a bug in -d/--def.
3. Changes to ntcoin and ntcoin_openbsd_tildeinstitute to check for the correct number of arguments.
4. Changes to ntcoin and ntcoin_openbsd_tildeinstitute to include the host name as a preprocessor constant.
|
| |
|
|
|
|
| |
1. Removed unneeded WHOAMI_PATH preprocessor constant from pcoin.cpp.
2. Updated WHOAMI_PATH for tilde.institute to use "id -ru" to get the real uid on OpenBSD (/usr/bin/whoami doesn't get the real username on OpenBSD).
|
| |
|
| |
I forgot that base_amount held the number of centitildecoins, and that I had to use cout_formatted_amount() to actually send to stdout the number of tildecoins (with decimal places).
|
| |
|
| |
I forgot to type "help(" and had just typed "void long long int base_amount)".
|
| |
|
| |
Brought to my attention by ~mastergeek on tilde.team, the tcoin help incorrectly said that the number of tildecoins a user started with was 1000. It was actually the current base amount (unaltered by other sources like minercoin, tildebot or daily-adventure). Now, the help shows the right amount of starting coins.
|
| |
|
|
| |
Thanks to ~lickthecheese on tilde.town, tilde.team and other tildeboxes, I was informed of this typo: 'creata' -> 'create'
|
| |
|
|
|
| |
And I don't know how to clean my commits after having pushed
them to remote origin/master. Maybe a Git stash and hard reset?
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
After having received a 10000-character transaction message from ~jan6@envs.net,
I finally came around to implementing character limits on transaction messages.
Previously, I was of the opinion that one should be able to send a movie as a
transaction message if one so wished.
I took inspiration for an appropriate character limit from Australia's NPP (New
Payments Platform), which has a 280-character limit on transaction messages. I
strongly suspect this limit came from Twitter's analysis of tweets in different
languages and Twitter research that concluded that a 280-character tweet was the
appropriate length of tweet. A transaction message is supposedly like a tweet with
some money sent along with it.
|
| |\
| |
| |
| |
| | |
I made a mistake. I should have done this before making my commits.
That's the only reason why this merge is necessary.
|
| | |
| |
| |
| |
| |
| | |
I had forgotten to change \u23CE to <new-line> and introduce a check for <vertical-tab> in many many places
in tcoin.cpp and pcoin.cpp. I should probably put these things in some kind of function so I don't repeat myself.
I'll probably do that in a future commit.
|
| |/
|
|
|
|
| |
constants
I had forgotten to handle the fatal errors and take the error numbers out into symbolic constants. I have now done that in tcoin.cpp and pcoin.cpp
|
| |
|
|
| |
This doesn't change functionality. It is simply a refactor that separates error codes out into symbolic constants in tcoin.cpp and pcoin.cpp.
|
| |
|
|
|
|
| |
<vertical-tab>
Newlines and vertical tabs in messages are replaced with <new-line> and <vertical-tab> to prevent transaction spoofing inside a message.
|
| |
|
|
| |
Just added a period/fullstop to the message in the show_balances() function.
|
| |
|
|
| |
The title is confusing, sorry. "No messages found" is shown in tcoin and pcoin when no messages are found (and this no-message message has one newline above and one newline below it instead of two newlines above it as before).
|
| |
|
|
| |
Just a minor aesthetic fix so that there is even newline gaps between statements when running 'tcoin init'.
|
| |
|
|
|
|
|
|
|
|
|
| |
Several changes were made that were difficult to separate into separate commits. Since features are directly committed to master, there are no feature branches.
The high-level list of changes is as follows:
1. Fixed a bug in pcoin where using "pcoin send -s" for silentsend would forget to add the transaction message that provides a record of who transferred how much and when, even if there is no custom message appended (send_message(.) was not called).
2. Improved the help text to make it clear that sending messages could be done using silentsend too.
3. In several cases in tcoin and pcoin, certain ways of doing a silentsend would still send with verbose output because, by mistake, the option passed to send_message(.) was not changed from "verbose" to "silent".
4. In pcoin, there was the case of 6 arguments missing, where one does pcoin send -s <user> <amount> <message>. This case was handled in tcoin but not pcoin. Now, it is handled in pcoin too.
5. Now, when a custom message is sent in tcoin or pcoin, an additional text confirming the successful sending of the custom message as distinct from the one confirming the sending of the coins is displayed to the user.
|
| |
|
|
|
|
| |
The KROWBAR_OFF, DA_OFF and MINERCOIN_OFF symbolic constants are now included in tcoin_defs.cpp and pcoin_defs.cpp and have been removed from the main source files, tcoin.cpp and pcoin.cpp.
ntcoin has been modified to include #define KROWBAR_OFF, #define DA_OFF and #define MINERCOIN_OFF in pcoin_defs.cpp and tcoin_defs.cpp by default, since those three features are only (as of now) available on the tilde.town server and on no other server.
|
| |
|
|
|
|
| |
I realised that different tilde servers will have different definitions but the same code otherwise, and 'git push' and 'git pull' to update the code on other servers would override those definitions.
Thus, the tcoin.cpp and pcoin.cpp files now look the same for all tildeservers and have a #include "tcoin_defs.cpp" and #include "pcoin_defs.cpp" respectively. The configuration, which is in the form of #define symbolic constants, is stored in tcoin/bin/tcoin_defs.cpp and tcoin/bin/pcoin_defs.cpp. ntcoin is modified to create these two files with the right definitions. ntcoin is also modified so that compile_tcoin and compile_pcoin now include the -I option, which instructs the compiler to look for includes in additional directories, in this case, tcoin/bin .
|
| |
|
|
| |
Benjamin Harris (https://ben.tild3.org, https://github.com/benharri, and https://benharr.is) found a bug where the time recorded for transaction logs/messages was local server time instead of UTC. It should have been UTC no matter what the local time was.
|
| |
|
|
| |
To fix the preprocessor constants so that they point to the right directory for the tcoin and pcoin executables. This will be different depending on where the tcoin and pcoin executables are stored.
|
| |
|
|
| |
A default message is now shown when there are no messages to show.
|
| |
|
|
| |
pcoin.cpp and tcoin.cpp were missing the preprocessor directives generated by ntcoin. The right preprocessor directives are now included in both files.
|
| |
|
|
|
|
| |
integrations
Defining the symbolic constants NO_KROWBAR, NO_DA and NO_MINERCOIN disables the three coin integrations during compile time.
|
| |
|
| |
Updating the successful login message to add additional information about running `tcoin` to check one's balance and messages. This is intended to make it easy for new users to know what to do once they've done "tcoin on".
|
| |
|
| |
When there are fewer than <n> messages in total, and one runs "tcoin -m <n>", an extra newline was shown between "Last <n> messages:" and the actual messages. This commit fixes that by checking if the first character in the actual messages is a newline and putting one more newline only if it isn't (instead of always).
|
| |
|
| |
Fixed bug in "tcoin init" and "tcoin" when an uninitialised user runs them. Previously, it used to display the following error: Error! Could not open file at <file_path>! Assuming its internal content is "0\n" because 'user_amount' in 'int main()' was read from the file before checking if the user had initialised.
|
| |
|
| |
credit to eigenellies/diodelasses and npa
|
| |
|
| |
daily-adventure changed how it stores state, so this new code fixes daily-adventure integration to the new format that daily-adventure stores state in.
|
| | |
|
| |
|