disable help and guestbook; minor fixes

fix validation REs, fix key thing
upgrade django
2021-03-08 01:14:36 +00:00 · 2021-03-07 21:52:26 +00:00 · 2020-06-03 02:03:51 +00:00 · 2019-12-18 14:07:40 +00:00 · 2019-12-16 22:25:20 +00:00 · 2019-12-16 22:25:03 +00:00
10 changed files with 44 additions and 17 deletions
--- a/scripts/start.sh
+++ b/scripts/start.sh
@ -9,4 +9,4 @@ VENV=/town/venvs/ttadmin
 source $VENV/bin/activate
 export DJANGO_SETTINGS_MODULE=settings_live
 cd $APP_ROOT
-gunicorn -b0.0.0.0:$PORT ttadmin.wsgi
+gunicorn -t120 -b0.0.0.0:$PORT ttadmin.wsgi
--- a/setup.py
+++ b/setup.py
@ -15,9 +15,9 @@ setup(
        'License :: OSI Approved :: Affero GNU General Public License v3 (AGPLv3)',
    ],
    packages=['ttadmin'],
-    install_requires = ['Django==1.10.2',
+    install_requires = ['Django==1.11.29',
                        'sshpubkeys==2.2.0',
-                        'psycopg2==2.7.6.1',
+                        'psycopg2-binary==2.8.5',
                        'gunicorn==19.6.0',
                        'Mastodon.py==1.4.5',
                        'tweepy==3.7.0'],
--- a/ttadmin/guestbook/templates/guestbook/guestbook.html
+++ b/ttadmin/guestbook/templates/guestbook/guestbook.html
@ -20,15 +20,15 @@
    </head>
    <body>
        <h1>tilde.town guestbook</h1>
+        <p><em>don't try to post urls. it won't work.</em></p>
        <marquee>~*~*~*~*say hello*~*~*~*~</marquee>
-        <!-- <form class="tilde" action="{% url 'guestbook:guestbook' %}" method="post">
+        <form class="tilde" action="{% url 'guestbook:guestbook' %}" method="post">
            {% csrf_token %}
            <table>
                {{form.as_table}}
            </table>
            <input type="submit" value="sign" >
        </form>
-        -->

        {% for m in messages %}
        <p>
--- a/ttadmin/guestbook/views.py
+++ b/ttadmin/guestbook/views.py
@ -1,3 +1,5 @@
+import re
+
 from django.shortcuts import redirect
 from django.views.generic import TemplateView
 from django.views.generic.edit import FormView
@ -5,6 +7,8 @@ from django.views.generic.edit import FormView
 from .forms import GuestbookForm
 from .models import GuestbookMessage

+SUSPICIOUS_RE = re.compile(r'https?://')
+

 class GuestbookView(FormView):
    form_class = GuestbookForm
@ -17,5 +21,7 @@ class GuestbookView(FormView):

    def form_valid(self, form):
        del form.cleaned_data['captcha']
+        if SUSPICIOUS_RE.search(form.cleaned_data['msg']) != None:
+            return redirect('guestbook:guestbook')
        t = GuestbookMessage.objects.create(**form.cleaned_data)
        return redirect('guestbook:guestbook')
--- a/ttadmin/urls.py
+++ b/ttadmin/urls.py
@ -2,8 +2,8 @@ from django.conf.urls import url, include
 from django.contrib import admin

 urlpatterns = [
-    url(r'^help/', include('help.urls')),
+    # url(r'^help/', include('help.urls')),
    url(r'^users/', include('users.urls')),
-    url(r'^guestbook/', include('guestbook.urls')),
+    # url(r'^guestbook/', include('guestbook.urls')),
    url(r'^admin/', admin.site.urls),
 ]
--- a/ttadmin/users/admin.py
+++ b/ttadmin/users/admin.py
@ -11,7 +11,7 @@ class PubkeyInline(admin.TabularInline):

 def bulk_accept(madmin, req, qs):
    for townie in qs:
-        townie.state = 'accepted'
+        townie.state = Townie.ACCEPTED
        townie.save()
    post_users_to_social(qs)

@ -19,7 +19,7 @@ bulk_accept.short_description = 'mark selected townies as accepted'

 def bulk_reject(madmin, req, qs):
    for townie in qs:
-        townie.state = 'rejected'
+        townie.state = Townie.REJECTED
        townie.save()

 bulk_reject.short_description = 'mark selected townies as rejected'
--- a/ttadmin/users/forms.py
+++ b/ttadmin/users/forms.py
@ -12,9 +12,9 @@ submission_throttle = {}
 throttle_submission = throttler(submission_throttle)


-USERNAME_RE = re.compile(r'[a-z][a-z0-9_]+')
-USERNAME_MIN_LENGTH = 3
-DISPLAY_NAME_RE = re.compile(r"[a-zA-Z0-9_\-']+")
+USERNAME_RE = re.compile(r'^[a-z][a-z0-9_]+$')
+USERNAME_MIN_LENGTH = 2
+DISPLAY_NAME_RE = re.compile(r"^[a-zA-Z0-9_\-']+$")
 DISPLAY_MIN_LENGTH = 2


--- a/ttadmin/users/models.py
+++ b/ttadmin/users/models.py
@ -19,6 +19,7 @@ SSH_TYPE_CHOICES = (
    ('ssh-rsa', 'ssh-rsa',),
    ('ssh-dss', 'ssh-dss',),
    ('ecdsa-sha2-nistp256', 'ecdsa-sha2-nistp256'),
+    ('ssh-ed25519', 'ssh-ed25519'),
 )

 DEFAULT_INDEX_PATH = '/etc/skel/public_html/index.html'
@ -78,6 +79,13 @@ class Townie(User):
    def home(self):
        return os.path.join('/home', self.username)

+    def generate_gift(self):
+        command = '/town/bin/generate_welcome_present.sh'
+        error = _guarded_run(['sudo', command, self.username])
+        if error:
+            logger.error(error)
+            return
+
    def send_welcome_email(self):
        welcome_tmpl = get_template('users/welcome_email.txt')
        context = {
@ -246,9 +254,15 @@ def on_townie_pre_save(sender, instance, **kwargs):
    # See if we need to create the user on disk.
    if existing.unreviewed and instance.accepted:
        logger.info('Creating user {} on disk.'.format(instance.username))
-        instance.create_on_disk()
-        instance.send_welcome_email()
-        instance.write_authorized_keys()
+        try:
+            instance.create_on_disk()
+            instance.write_authorized_keys()
+        except Exception as e:
+            logger.error('Failed syncing user {} to disk: {}'.format(instance.username, e))
+        else:
+            instance.send_welcome_email()
+            instance.generate_gift()
+
        return
    else:
        # This user state transition is currently undefined. In the future, we can check for things
--- a/ttadmin/users/templates/users/thanks.html
+++ b/ttadmin/users/templates/users/thanks.html
@ -27,7 +27,7 @@
                        <li><a href="https://tilde.town/~kc/blackout/">black out</a></li>
                        <li><a href="https://tilde.town/~subtransience/machinecode/index.html">the machine room</a></li>
                    </ul>
-                    <h2>or a <a href="https://tilde.town/cgi/random">random page</a></h2>
+                    <h2>or a <a href="https://cgi.tilde.town/users/random">random page</a></h2>
        </td>
        <td style="padding:1em">
            <a href="http://giphy.com/gifs/cyndipop-golden-girls-bea-arthur-ToMjGpK80QLT7KLWPLO">
--- a/ttadmin/users/views.py
+++ b/ttadmin/users/views.py
@ -12,10 +12,17 @@ from django.views.generic.edit import FormView
 from .forms import TownieForm
 from .models import Townie, Pubkey

+SIGNUPS_ENABLED = True
+
 class SignupView(FormView):
    form_class = TownieForm
    template_name = 'users/signup.html'
-    extra_context = {"signups_enabled": False}
+
+    def get_context_data(self, **kwargs):
+        ctx = super().get_context_data(**kwargs)
+        ctx['signups_enabled'] = SIGNUPS_ENABLED
+        return ctx
+

    @transaction.atomic
    def form_valid(self, form):
Author	SHA1	Message	Date
vilmibm	afc5d30ed8	disable help and guestbook; minor fixes	2021-03-08 01:14:36 +00:00
vilmibm	8d04787277	fix validation REs, fix key thing	2021-03-07 21:52:26 +00:00
vilmibm	be2975ad62	upgrade django	2020-06-03 02:03:51 +00:00
vilmibm	0f078f86fd	upgrade Django	2019-12-18 14:07:40 +00:00
vilmibm	90dca0274a	return to generating gifts	2019-12-16 22:25:20 +00:00
vilmibm	158bdcff5f	guestbook spam stuff	2019-12-16 22:25:03 +00:00
vilmibm	77d728a153	bootleg anti-spam measure	2019-12-16 22:22:01 +00:00
vilmibm	2385446e74	increase timeout to help with bulk add of users	2019-12-16 22:21:44 +00:00
vilmibm	5220d9bfad	atomic user creation/notification	2019-10-11 22:53:39 +00:00
vilmibm	4e3977edd2	enum sigh	2019-08-21 15:39:49 +00:00
vilmibm	2354695401	fix signups toggle	2019-08-20 21:39:38 +00:00