aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMike Macgirvin <mike@macgirvin.com>2010-07-22 02:13:39 -0700
committerMike Macgirvin <mike@macgirvin.com>2010-07-22 02:13:39 -0700
commit11f2eaa16dc3e41b758235e7bc1a281e19aca96e (patch)
tree3d2aff70c4efa8c51f13c3942bf6f65f1c9ad409
parenta9befec59e2e608b507e6d1f99a5a2fe70978c3f (diff)
downloadvolse-hubzilla-11f2eaa16dc3e41b758235e7bc1a281e19aca96e.tar.gz
volse-hubzilla-11f2eaa16dc3e41b758235e7bc1a281e19aca96e.tar.bz2
volse-hubzilla-11f2eaa16dc3e41b758235e7bc1a281e19aca96e.zip
introduction cleanup - email notification flags
-rw-r--r--boot.php9
-rw-r--r--mod/dfrn_request.php265
-rw-r--r--update.sql38
-rw-r--r--view/dfrn_request.tpl2
4 files changed, 187 insertions, 127 deletions
diff --git a/boot.php b/boot.php
index 5efa9a24f..b36a255eb 100644
--- a/boot.php
+++ b/boot.php
@@ -13,6 +13,15 @@ define ( 'DIRECTION_IN', 1);
define ( 'DIRECTION_OUT', 2);
define ( 'DIRECTION_BOTH', 3);
+define ( 'NOTIFY_INTRO', 0x0001 );
+define ( 'NOTIFY_CONFIRM', 0x0002 );
+define ( 'NOTIFY_WALL', 0x0004 );
+define ( 'NOTIFY_COMMENT', 0x0008 );
+define ( 'NOTIFY_MAIL', 0x0010 );
+
+
+
+
if(! class_exists('App')) {
class App {
diff --git a/mod/dfrn_request.php b/mod/dfrn_request.php
index 4b41c9c6c..35ab657e1 100644
--- a/mod/dfrn_request.php
+++ b/mod/dfrn_request.php
@@ -3,11 +3,6 @@
if(! function_exists('dfrn_request_init')) {
function dfrn_request_init(&$a) {
- if($_SESSION['authenticated']) {
- // choose which page to show (could be remote auth)
-
- }
-
if($a->argc > 1)
$which = $a->argv[1];
@@ -30,127 +25,145 @@ function dfrn_request_post(&$a) {
}
- // callback to local site after remote request and local confirm
+ // We've introduced ourself to another cell, then have been returned to our own cell
+ // to confirm the request, and then we've clicked submit (perhaps after logging in).
+ // That brings us here:
- if((x($_POST,'localconfirm')) && ($_POST['localconfirm'] == 1)
- && local_user() && ($a->user['nickname'] == $a->argv[1]) && (x($_POST,'dfrn_url'))) {
+ if((x($_POST,'localconfirm')) && ($_POST['localconfirm'] == 1)) {
- // We are the requestor, and we've been sent back to our own site
- // to confirm the request. We've done so and clicked submit,
- // which brings us here.
+ // Ensure this is a valid request
+
+ if(local_user() && ($a->user['nickname'] == $a->argv[1]) && (x($_POST,'dfrn_url'))) {
- $dfrn_url = notags(trim($_POST['dfrn_url']));
- $aes_allow = (((x($_POST,'aes_allow')) && ($_POST['aes_allow'] == 1)) ? 1 : 0);
- $confirm_key = ((x($_POST,'confirm_key')) ? $_POST['confirm_key'] : "");
- $contact_record = null;
-
- if(x($dfrn_url)) {
+ $dfrn_url = notags(trim($_POST['dfrn_url']));
+ $aes_allow = (((x($_POST,'aes_allow')) && ($_POST['aes_allow'] == 1)) ? 1 : 0);
+ $confirm_key = ((x($_POST,'confirm_key')) ? $_POST['confirm_key'] : "");
- $r = q("SELECT * FROM `contact` WHERE `uid` = %d AND `url` = '%s' LIMIT 1",
- intval($_SESSION['uid']),
- dbesc($dfrn_url)
- );
+ $contact_record = null;
- if(count($r)) {
- if(strlen($r[0]['dfrn-id'])) {
- notice("This introduction has already been accepted." . EOL );
- return;
- }
- else
- $contact_record = $r[0];
- }
+ if(x($dfrn_url)) {
- if(is_array($contact_record)) {
- $r = q("UPDATE `contact` SET `ret-aes` = %d WHERE `id` = %d LIMIT 1",
- intval($aes_allow),
- intval($contact_record['id'])
+ $r = q("SELECT * FROM `contact` WHERE `uid` = %d AND `url` = '%s' LIMIT 1",
+ intval($_SESSION['uid']),
+ dbesc($dfrn_url)
);
- }
- else {
-
- require_once('Scrape.php');
-
-
- $parms = scrape_dfrn($dfrn_url);
-
- if(! count($parms)) {
- notice( 'Profile location is not valid or does not contain profile information.' . EOL );
- return;
+
+ if(count($r)) {
+ if(strlen($r[0]['dfrn-id'])) {
+ notice("This introduction has already been accepted." . EOL );
+ return;
+ }
+ else
+ $contact_record = $r[0];
+ }
+
+ if(is_array($contact_record)) {
+ $r = q("UPDATE `contact` SET `ret-aes` = %d WHERE `id` = %d LIMIT 1",
+ intval($aes_allow),
+ intval($contact_record['id'])
+ );
}
else {
- if(! x($parms,'fn'))
- notice( 'Warning: profile location has no identifiable owner name.' . EOL );
- if(! x($parms,'photo'))
- notice( 'Warning: profile location has no profile photo.' . EOL );
- $invalid = validate_dfrn($parms);
- if($invalid) {
- notice( $invalid . ' required parameter'
- . (($invalid == 1) ? " was " : "s were " )
- . "not found at the given location." . EOL ) ;
+
+ require_once('Scrape.php');
+
+
+ $parms = scrape_dfrn($dfrn_url);
+
+ if(! count($parms)) {
+ notice( 'Profile location is not valid or does not contain profile information.' . EOL );
return;
}
- }
+ else {
+ if(! x($parms,'fn'))
+ notice( 'Warning: profile location has no identifiable owner name.' . EOL );
+ if(! x($parms,'photo'))
+ notice( 'Warning: profile location has no profile photo.' . EOL );
+ $invalid = validate_dfrn($parms);
+ if($invalid) {
+ notice( $invalid . ' required parameter'
+ . (($invalid == 1) ? " was " : "s were " )
+ . "not found at the given location." . EOL ) ;
+ return;
+ }
+ }
- $dfrn_request = $parms['dfrn-request'];
+ $dfrn_request = $parms['dfrn-request'];
- dbesc_array($parms);
+ dbesc_array($parms);
- $r = q("INSERT INTO `contact` ( `uid`, `created`,`url`, `name`, `photo`, `site-pubkey`,
- `request`, `confirm`, `notify`, `poll`, `aes_allow`)
- VALUES ( %d, '%s', '%s', '%s' , '%s', '%s', '%s', '%s', '%s', '%s', %d)",
- intval($_SESSION['uid']),
- datetime_convert(),
- dbesc($dfrn_url),
- $parms['fn'],
- $parms['photo'],
- $parms['key'],
- $parms['dfrn-request'],
- $parms['dfrn-confirm'],
- $parms['dfrn-notify'],
- $parms['dfrn-poll'],
- intval($aes_allow)
- );
- }
+ $r = q("INSERT INTO `contact` ( `uid`, `created`,`url`, `name`, `photo`, `site-pubkey`,
+ `request`, `confirm`, `notify`, `poll`, `aes_allow`)
+ VALUES ( %d, '%s', '%s', '%s' , '%s', '%s', '%s', '%s', '%s', '%s', %d)",
+ intval($_SESSION['uid']),
+ datetime_convert(),
+ dbesc($dfrn_url),
+ $parms['fn'],
+ $parms['photo'],
+ $parms['key'],
+ $parms['dfrn-request'],
+ $parms['dfrn-confirm'],
+ $parms['dfrn-notify'],
+ $parms['dfrn-poll'],
+ intval($aes_allow)
+ );
+ }
- if($r) {
- notice( "Introduction complete." . EOL);
- }
+ if($r) {
+ notice( "Introduction complete." . EOL);
+ }
+
+ // Allow the blocked remote notification to complete
- // Allow the blocked remote notification to complete
+ if(is_array($contact_record))
+ $dfrn_request = $contact_record['request'];
- if(is_array($contact_record))
- $dfrn_request = $contact_record['request'];
+ if(strlen($dfrn_request) && strlen($confirm_key))
+ $s = fetch_url($dfrn_request . '?confirm_key=' . $confirm_key);
+ // ignore reply
+ goaway($dfrn_url);
+ return; // NOTREACHED
- if(strlen($dfrn_request) && strlen($confirm_key))
- $s = fetch_url($dfrn_request . '?confirm_key=' . $confirm_key);
- // ignore reply
- goaway($dfrn_url);
- // NOTREACHED
+ }
}
- // invalid DFRN-url
+
+ // invalid/bogus request
+
notice( "Unrecoverable protocol error." . EOL );
goaway($a->get_baseurl());
+ return; // NOTREACHED
}
- // extra safety
- if($_POST['localconfirm'])
- return;
+ // Otherwise:
+
+ // We are the requestee. A person from a remote cell has made an introduction
+ // on our profile web page and clicked submit. We will use their DFRN-URL to
+ // figure out how to contact their cell.
- // we are operating as a remote site and an introduction was requested of us.
// Scrape the originating DFRN-URL for everything we need. Create a contact record
// and an introduction to show our user next time he/she logs in.
- // Finally redirect back to the originator so that their site can record the request.
- // If our user confirms the request, a record of it will need to exist on the
- // originator's site in order for the confirmation process to complete..
+ // Finally redirect back to the requestor so that their site can record the request.
+ // If our user (the requestee) later confirms this request, a record of it will need
+ // to exist on the requestor's cell in order for the confirmation process to complete..
+ // It's possible that neither the requestor or the requestee are logged in at the moment,
+ // and the requestor does not yet have any credentials to the requestee profile.
- $tailname = $a->profile['nickname'];
+ // Who is the requestee? We've already loaded their profile which means their nickname should be
+ // in $a->argv[1] and we should have their complete info in $a->profile.
+ if(! (is_array($a->profile) && count($a->profile))) {
+ notice("Profile unavailable." . EOL);
+ return;
+ }
+
+ $nickname = $a->profile['nickname'];
+ $notify_flags = $a->profile['notify-flags'];
$uid = $a->profile['uid'];
$contact_record = null;
@@ -166,6 +179,8 @@ function dfrn_request_post(&$a) {
return;
}
+ // Is this an email-style DFRN locator?
+
if(strstr($url,'@')) {
$username = substr($url,0,strpos($url,'@'));
$hostname = substr($url,strpos($url,'@') + 1);
@@ -297,23 +312,27 @@ function dfrn_request_post(&$a) {
);
}
- // TODO: send an email notification if our user wants one
+
+ // This notice will only be seen by the requestor if the requestor and requestee are on the same server.
if(! $failed)
notice( "Your introduction has been sent." . EOL );
// "Homecoming" - send the requestor back to their site to record the introduction.
- $dfrn_url = bin2hex($a->get_baseurl() . "/profile/$tailname");
+ $dfrn_url = bin2hex($a->get_baseurl() . "/profile/$nickname");
$aes_allow = ((function_exists('openssl_encrypt')) ? 1 : 0);
goaway($parms['dfrn-request'] . "?dfrn_url=$dfrn_url" . '&confirm_key=' . $hash . (($aes_allow) ? "&aes_allow=1" : ""));
- // NOTREACHED
+ return; // NOTREACHED
}
return;
}}
+
+
+
if(! function_exists('dfrn_request_content')) {
function dfrn_request_content(&$a) {
@@ -337,12 +356,12 @@ function dfrn_request_content(&$a) {
// Edge case, but can easily happen in the wild. This person is authenticated,
// but not as the person who needs to deal with this request.
- if (($_SESSION['uid'] != $a->argv[1]) && ($a->user['nickname'] != $a->argv[1])) {
+ if ($a->user['nickname'] != $a->argv[1]) {
notice( "Incorrect identity currently logged in. Please login to <strong>this</strong> profile." . EOL);
return login();
}
- $dfrn_url = notags(trim(pack("H*" , $_GET['dfrn_url'])));
+ $dfrn_url = notags(trim(hex2bin($_GET['dfrn_url'])));
$aes_allow = (((x($_GET,'aes_allow')) && ($_GET['aes_allow'] == 1)) ? 1 : 0);
$confirm_key = (x($_GET,'confirm_key') ? $_GET['confirm_key'] : "");
$o .= file_get_contents("view/dfrn_req_confirm.tpl");
@@ -358,21 +377,53 @@ function dfrn_request_content(&$a) {
return $o;
}
- else {
- // we are the requestee and it is now safe to send our user their introduction
- if((x($_GET,'confirm_key')) && strlen($_GET['confirm_key'])) {
+ elseif((x($_GET,'confirm_key')) && strlen($_GET['confirm_key'])) {
+
+ // we are the requestee and it is now safe to send our user their introduction,
+ // We could just unblock it, but first we have to jump through a few hoops to
+ // send an email, or even to find out if we need to send an email.
+
+ $intro = q("SELECT * FROM `intro` WHERE `hash` = '%s' LIMIT 1",
+ dbesc($_GET['confirm_key'])
+ );
+
+ if(count($intro)) {
+
+ $r = q("SELECT `contact`.*, `user`.* FROM `contact` LEFT JOIN `user` ON `contact`.`uid` = `user`.`uid`
+ WHERE `contact`.`id` = %d LIMIT 1",
+ intval($intro[0]['contact-id'])
+ );
+ if(count($r)) {
+
+ if($r[0]['notify-flags'] & NOTIFY_INTRO) {
+ $email_tpl = file_get_contents('view/request_notify_eml.tpl');
+ $email = replace_macros($email_tpl, array(
+ '$requestor' => ((strlen(stripslashes($r[0]['name']))) ? stripslashes($r[0]['name']) : '[Name Withheld]'),
+ '$url' => stripslashes($r[0]['url']),
+ '$myname' => $r[0]['username'],
+ '$siteurl' => $a->get_baseurl(),
+ '$sitename' => $a->config['sitename']
+ ));
+ $res = mail($r[0]['email'],"Introduction received at {$a->config['sitename']}",$email,"From: Administrator@{$_SERVER[SERVER_NAME]}");
+ // This is a redundant notification - no point throwing errors if it fails.
+ }
+ }
+
$r = q("UPDATE `intro` SET `blocked` = 0 WHERE `hash` = '%s' LIMIT 1",
dbesc($_GET['confirm_key'])
);
- killme();
- }
-
- // Outside request. Display our user's introduction form.
+ }
+ killme();
+ return; // NOTREACHED
+ }
+ else {
+ // Normal web request. Display our user's introduction form.
- $o = file_get_contents("view/dfrn_request.tpl");
- $o = replace_macros($o,array('$uid' => $a->argv[1]));
- return $o;
+ $o = file_get_contents("view/dfrn_request.tpl");
+ $o = replace_macros($o,array('$nickname' => $a->argv[1]));
+ return $o;
}
+ return; // Somebody is fishing.
}} \ No newline at end of file
diff --git a/update.sql b/update.sql
index a576adb88..ac11812cd 100644
--- a/update.sql
+++ b/update.sql
@@ -1,21 +1,21 @@
- ALTER TABLE `item` CHANGE `allow_uid` `allow_cid` MEDIUMTEXT CHARACTER SET utf8 COLLATE utf8_general_ci NOT NULL ,
-CHANGE `deny_uid` `deny_cid` MEDIUMTEXT CHARACTER SET utf8 COLLATE utf8_general_ci NOT NULL ;
+CREATE TABLE IF NOT EXISTS `mail` (
+ `id` int(10) unsigned NOT NULL,
+ `uid` int(10) unsigned NOT NULL,
+ `from-name` char(255) NOT NULL,
+ `from-photo` char(255) NOT NULL,
+ `from-url` char(255) NOT NULL,
+ `contact-id` char(255) NOT NULL,
+ `title` char(255) NOT NULL,
+ `body` text NOT NULL,
+ `delivered` tinyint(1) NOT NULL,
+ `seen` tinyint(1) NOT NULL,
+ `replied` tinyint(1) NOT NULL,
+ `uri` char(255) NOT NULL,
+ `parent-uri` char(255) NOT NULL,
+ `created` datetime NOT NULL DEFAULT '0000-00-00 00:00:00',
+ PRIMARY KEY (`id`)
+) ENGINE=MyISAM DEFAULT CHARSET=utf8;
- ALTER TABLE `item` CHANGE `last-child` `last-child` TINYINT( 1 ) UNSIGNED NOT NULL DEFAULT '1';
+ALTER TABLE `user` ADD `notify-flags` INT(11) UNSIGNED NOT NULL DEFAULT 65535 AFTER `blocked`;
-alter table `item` insert `remote-id` char( 255 ) character set utf-8 collate utf8_general_ci NOT NULL;
-
-ALTER TABLE `item` ADD `remote-name` CHAR( 255 ) CHARACTER SET utf8 COLLATE utf8_general_ci NOT NULL AFTER `remote-id` ,
-ADD `remote-link` CHAR( 255 ) CHARACTER SET utf8 COLLATE utf8_general_ci NOT NULL AFTER `remote-name` ,
-ADD `remote-avatar` CHAR( 255 ) CHARACTER SET utf8 COLLATE utf8_general_ci NOT NULL AFTER `remote-link` ;
-
-ALTER TABLE `item` ADD `owner-name` CHAR( 255 ) CHARACTER SET utf8 COLLATE utf8_general_ci NOT NULL AFTER `contact-id` ,
-ADD `owner-link` CHAR( 255 ) CHARACTER SET utf8 COLLATE utf8_general_ci NOT NULL AFTER `owner-name` ,
-ADD `owner-avatar` CHAR( 255 ) CHARACTER SET utf8 COLLATE utf8_general_ci NOT NULL AFTER `owner-link` ;
-
-ALTER TABLE `user` ADD `pwdreset` CHAR( 255 ) NOT NULL AFTER `blocked` ;
-
- ALTER TABLE `challenge` ADD `type` CHAR( 255 ) NOT NULL AFTER `expire` ,
-ADD `last_update` CHAR( 255 ) NOT NULL AFTER `type` ;
-
-ALTER TABLE `contact` ADD `last-update` DATETIME NOT NULL DEFAULT '0000-00-00 00:00:00' AFTER `ret-pubkey` ;
+--------------
diff --git a/view/dfrn_request.tpl b/view/dfrn_request.tpl
index c1a7b1d9b..483cf6a6f 100644
--- a/view/dfrn_request.tpl
+++ b/view/dfrn_request.tpl
@@ -6,7 +6,7 @@ You may introduce yourself to this member if you have a valid profile locator<br
on the <a href="http://dfrn.org">Distributed Friends and Relations Network (DFRN)</a>.
</p>
-<form action="dfrn_request/$uid" method="post" />
+<form action="dfrn_request/$nickname" method="post" />
<div id="dfrn-request-url-wrapper" >
<label id="dfrn-url-label" for="dfrn-url" >Your profile location:</label>