Skip to content

Commit 29cdfa8

Browse files
committed
Deleted some global vars and changed buffer to memory, #53
Signed-off-by: Sven Haardiek <Sven@Haardiek.de>
1 parent 49f129c commit 29cdfa8

File tree

1 file changed

+20
-27
lines changed

1 file changed

+20
-27
lines changed

ircrypt.py

Lines changed: 20 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -64,20 +64,16 @@
6464
import weechat, string, os, subprocess, base64, time
6565

6666

67-
# Global buffers used to store message parts, pending requests, configuration
67+
# Global memory used to store message parts, pending requests, configuration
6868
# options, keys, etc.
69-
ircrypt_msg_buffer = {}
69+
ircrypt_msg_memory = {}
7070
ircrypt_config_file = None
7171
ircrypt_config_section = {}
7272
ircrypt_config_option = {}
7373
ircrypt_keys = {}
7474
ircrypt_asym_id = {}
7575
ircrypt_cipher = {}
76-
ircrypt_buffer = None
77-
ircrypt_pending_requests = []
78-
ircrypt_request_buffer = {}
79-
ircrypt_pending_keys = []
80-
ircrypt_keys_buffer = {}
76+
ircrypt_keys_memory = {}
8177
ircrypt_gpg_binary = None
8278
ircrypt_message_plain = {}
8379
ircrypt_gpg_homedir = None
@@ -99,7 +95,6 @@
9995
%(bold)sIRCrypt command options: %(normal)s
10096
10197
list List set keys, ids and ciphers
102-
buffer Switch to/Open IRCrypt buffer
10398
set-key [-server <server>] <target> <key> Set key for target
10499
remove-key [-server <server>] <target> Remove key for target
105100
set-gpg-id [-server <server>] <nick> <id> Set public key identifier for nick
@@ -217,24 +212,23 @@ def ircrypt_public_key_send(server, args, info):
217212

218213

219214
def ircrypt_public_key_get(server, args, info):
220-
global ircrypt_keys_buffer, ircrypt_asym_id
215+
global ircrypt_keys_memory, ircrypt_asym_id
221216

222217
# Get prefix, number and message
223218
pre, message = args.split('>KCRY-', 1)
224219
number, message = message.split(' ', 1)
225220

226-
# Get key for the request buffer
227221
buf_key = (server, info['channel'], info['nick'])
228222

229223
# Check if we got the last part of the message otherwise put the message
230-
# into a global buffer and quit
224+
# into a global memory and quit
231225
if int(number):
232-
if not buf_key in ircrypt_keys_buffer:
226+
if not buf_key in ircrypt_keys_memory:
233227
# - First element is list of requests
234228
# - Second element is currently received request
235-
ircrypt_keys_buffer[buf_key] = MessageParts()
229+
ircrypt_keys_memory[buf_key] = MessageParts()
236230
# Add parts to current request
237-
ircrypt_keys_buffer[buf_key].update(int(number), message)
231+
ircrypt_keys_memory[buf_key].update(int(number), message)
238232
return ''
239233
else:
240234
target = ('%s/%s' % (server, info['nick'])).lower()
@@ -249,7 +243,7 @@ def ircrypt_public_key_get(server, args, info):
249243
'--import'], stdin=subprocess.PIPE, stdout=subprocess.PIPE,
250244
stderr=subprocess.PIPE)
251245
(out, err) = p.communicate(base64.b64decode(message +
252-
ircrypt_keys_buffer[buf_key].message))
246+
ircrypt_keys_memory[buf_key].message))
253247

254248
weechat.prnt('', err)
255249

@@ -273,8 +267,8 @@ def ircrypt_public_key_get(server, args, info):
273267

274268
# Set asymmetric identifier
275269
ircrypt_asym_id[target.lower()] = gpg_id
276-
# Print status message in current buffer
277270
weechat.prnt('', 'Set gpg key for %s' % target)
271+
# Print status message in current buffer
278272
ircrypt_sym_ex(server, info['nick'])
279273
return ''
280274

@@ -384,25 +378,25 @@ def ircrypt_decrypt_sym(server, args, info, key):
384378
:param info: dictionary created by info_get_hashtable
385379
:param key: key for decryption
386380
'''
387-
global ircrypt_msg_buffer, ircrypt_config_option
381+
global ircrypt_msg_memory, ircrypt_config_option
388382

389383
pre, message = string.split(args, '>CRY-', 1)
390384
number, message = string.split(message, ' ', 1 )
391385

392-
# Get key for the message buffer
386+
# Get key for the message memory
393387
buf_key = '%s.%s.%s' % (server, info['channel'], info['nick'])
394388

395389
# Decrypt only if we got last part of the message
396-
# otherwise put the message into a globa buffer and quit
390+
# otherwise put the message into a global memory and quit
397391
if int(number) != 0:
398-
if not buf_key in ircrypt_msg_buffer:
399-
ircrypt_msg_buffer[buf_key] = MessageParts()
400-
ircrypt_msg_buffer[buf_key].update(int(number), message)
392+
if not buf_key in ircrypt_msg_memory:
393+
ircrypt_msg_memory[buf_key] = MessageParts()
394+
ircrypt_msg_memory[buf_key].update(int(number), message)
401395
return ''
402396

403397
# Get whole message
404398
try:
405-
message = message + ircrypt_msg_buffer[buf_key].message
399+
message = message + ircrypt_msg_memory[buf_key].message
406400
except KeyError:
407401
pass
408402

@@ -432,9 +426,9 @@ def ircrypt_decrypt_sym(server, args, info, key):
432426
elif err:
433427
weechat.prnt(buf, '%s%s' % (weechat.color('gray'), err))
434428

435-
# Remove old messages from buffer
429+
# Remove old messages from memory
436430
try:
437-
del ircrypt_msg_buffer[buf_key]
431+
del ircrypt_msg_memory[buf_key]
438432
except KeyError:
439433
pass
440434
return '%s%s' % (pre, decrypted)
@@ -1087,7 +1081,6 @@ def ircrypt_key_generated_cb(data, command, errorcode, out, err):
10871081

10881082
weechat.hook_command('ircrypt', 'Manage IRCrypt Keys and public key identifier',
10891083
'[list] '
1090-
'| buffer '
10911084
'| set-key [-server <server>] <target> <key> '
10921085
'| remove-key [-server <server>] <target> '
10931086
'| request-public-key [-server <server>] <nick>'
@@ -1099,7 +1092,7 @@ def ircrypt_key_generated_cb(data, command, errorcode, out, err):
10991092
'| verify-keys [-server <server>] [<nick>] '
11001093
'| plain [-server <server>] [-channel <channel>] <message>',
11011094
ircrypt_help_text,
1102-
'list || buffer || set-key %(irc_channel)|%(nicks)|-server %(irc_servers) %- '
1095+
'list || set-key %(irc_channel)|%(nicks)|-server %(irc_servers) %- '
11031096
'|| remove-key %(irc_channel)|%(nicks)|-server %(irc_servers) %- '
11041097
'|| exchange %(nicks) %(irc_channel) -server %(irc_servers)'
11051098
'|| verify-requests %(nicks)|-server %(irc_servers) %- '

0 commit comments

Comments
 (0)