Rename save_self_keypair

For the ffi rename to dc_preconfigure_keypair.  For the internal API
to store_self_keypair.
This commit is contained in:
Floris Bruynooghe
2020-02-06 21:32:17 +01:00
committed by Floris Bruynooghe
parent a903805cd9
commit fc0292bf8a
8 changed files with 19 additions and 19 deletions

View File

@@ -870,7 +870,7 @@ void dc_maybe_network (dc_context_t* context);
* @param secret_data The secret key as base64. * @param secret_data The secret key as base64.
* @return 1 on success, 0 on failure. * @return 1 on success, 0 on failure.
*/ */
int _dc_save_self_keypair (dc_context_t* context, const char *addr, const char *public_data, const char *secret_data); int dc_preconfigure_keypair (dc_context_t* context, const char *addr, const char *public_data, const char *secret_data);
// handle chatlists // handle chatlists

View File

@@ -691,7 +691,7 @@ pub unsafe extern "C" fn dc_maybe_network(context: *mut dc_context_t) {
} }
#[no_mangle] #[no_mangle]
pub unsafe extern "C" fn _dc_save_self_keypair( pub unsafe extern "C" fn dc_preconfigure_keypair(
context: *mut dc_context_t, context: *mut dc_context_t,
addr: *const libc::c_char, addr: *const libc::c_char,
public_data: *const libc::c_char, public_data: *const libc::c_char,
@@ -712,7 +712,7 @@ pub unsafe extern "C" fn _dc_save_self_keypair(
public, public,
secret, secret,
}; };
key::save_self_keypair(ctx, &keypair, key::KeyPairUse::Default)?; key::store_self_keypair(ctx, &keypair, key::KeyPairUse::Default)?;
Ok(1) Ok(1)
}) })
.log_warn(ffi_context, "Failed to save keypair") .log_warn(ffi_context, "Failed to save keypair")

View File

@@ -118,12 +118,12 @@ class Account(object):
assert res != ffi.NULL, "config value not found for: {!r}".format(name) assert res != ffi.NULL, "config value not found for: {!r}".format(name)
return from_dc_charpointer(res) return from_dc_charpointer(res)
def _save_self_keypair(self, addr, public, secret): def _preconfigure_keypair(self, addr, public, secret):
"""See _dc_save_self_keypair() in deltachat.h. """See _dc_save_self_keypair() in deltachat.h.
In other words, you don't need this. In other words, you don't need this.
""" """
res = lib._dc_save_self_keypair(self._dc_context, res = lib.dc_preconfigure_keypair(self._dc_context,
as_dc_charpointer(addr), as_dc_charpointer(addr),
as_dc_charpointer(public), as_dc_charpointer(public),
as_dc_charpointer(secret)) as_dc_charpointer(secret))

View File

@@ -35,9 +35,9 @@ class TestOfflineAccountBasic:
ac1 = Account(p.strpath, os_name="solarpunk") ac1 = Account(p.strpath, os_name="solarpunk")
ac1.get_info() ac1.get_info()
def test_save_self_keypair(self, acfactory, datadir): def test_preconfigure_keypair(self, acfactory, datadir):
ac = acfactory.get_unconfigured_account() ac = acfactory.get_unconfigured_account()
ac._save_self_keypair("alice@example.com", ac._preconfigure_keypair("alice@example.com",
datadir.join('key/alice-public.asc').read(), datadir.join('key/alice-public.asc').read(),
datadir.join('key/alice-secret.asc').read()) datadir.join('key/alice-secret.asc').read())

View File

@@ -215,7 +215,7 @@ fn load_or_generate_self_public_key(
"Generating keypair with {} bits, e={} ...", 2048, 65537, "Generating keypair with {} bits, e={} ...", 2048, 65537,
); );
let keypair = pgp::create_keypair(EmailAddress::new(self_addr.as_ref())?)?; let keypair = pgp::create_keypair(EmailAddress::new(self_addr.as_ref())?)?;
key::save_self_keypair(context, &keypair, KeyPairUse::Default)?; key::store_self_keypair(context, &keypair, KeyPairUse::Default)?;
info!( info!(
context, context,
"Keypair generated in {:.3}s.", "Keypair generated in {:.3}s.",

View File

@@ -324,7 +324,7 @@ fn set_self_key(
public, public,
secret, secret,
}; };
key::save_self_keypair( key::store_self_keypair(
context, context,
&keypair, &keypair,
if set_default { if set_default {

View File

@@ -335,7 +335,7 @@ impl SaveKeyError {
} }
} }
/// Save the keypair as an owned keypair for addr. /// Store the keypair as an owned keypair for addr in the database.
/// ///
/// This will save the keypair as keys for the given address. The /// This will save the keypair as keys for the given address. The
/// "self" here refers to the fact that this DC instance owns the /// "self" here refers to the fact that this DC instance owns the
@@ -347,7 +347,7 @@ impl SaveKeyError {
/// same key again overwrites it. /// same key again overwrites it.
/// ///
/// [Config::ConfiguredAddr]: crate::config::Config::ConfiguredAddr /// [Config::ConfiguredAddr]: crate::config::Config::ConfiguredAddr
pub fn save_self_keypair( pub fn store_self_keypair(
context: &Context, context: &Context,
keypair: &KeyPair, keypair: &KeyPair,
default: KeyPairUse, default: KeyPairUse,
@@ -590,9 +590,9 @@ i8pcjGO+IZffvyZJVRWfVooBJmWWbPB1pueo3tx8w3+fcuzpxz+RLFKaPyqXO+dD
.unwrap() .unwrap()
}; };
assert_eq!(nrows(), 0); assert_eq!(nrows(), 0);
save_self_keypair(&t.ctx, &KEYPAIR, KeyPairUse::Default).unwrap(); store_self_keypair(&t.ctx, &KEYPAIR, KeyPairUse::Default).unwrap();
assert_eq!(nrows(), 1); assert_eq!(nrows(), 1);
save_self_keypair(&t.ctx, &KEYPAIR, KeyPairUse::Default).unwrap(); store_self_keypair(&t.ctx, &KEYPAIR, KeyPairUse::Default).unwrap();
assert_eq!(nrows(), 1); assert_eq!(nrows(), 1);
} }
} }

View File

@@ -85,7 +85,7 @@ pub(crate) fn configure_alice_keypair(ctx: &Context) -> String {
let keypair = alice_keypair(); let keypair = alice_keypair();
ctx.set_config(Config::ConfiguredAddr, Some(&keypair.addr.to_string())) ctx.set_config(Config::ConfiguredAddr, Some(&keypair.addr.to_string()))
.unwrap(); .unwrap();
key::save_self_keypair(&ctx, &keypair, key::KeyPairUse::Default) key::store_self_keypair(&ctx, &keypair, key::KeyPairUse::Default)
.expect("Failed to save Alice's key"); .expect("Failed to save Alice's key");
keypair.addr.to_string() keypair.addr.to_string()
} }