Skip to content

Commit 63d47cf

Browse files
committed
merging fix for missing id's on all inputs
2 parents 235a0ce + e7c423c commit 63d47cf

File tree

4 files changed

+22
-8
lines changed

4 files changed

+22
-8
lines changed

doc/upgrading.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -528,7 +528,7 @@ to::
528528
self.form_wins = True
529529
self.add_error_message(' '.join(msg.args))
530530

531-
This escapes the Referer value an prevents it from being
531+
This escapes the Referer value and prevents it from being
532532
executed.
533533

534534
.. _CVE-2024-39126:

roundup/cgi/client.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1677,8 +1677,8 @@ def handle_csrf(self, api=False):
16771677
if 'HTTP_X_REQUESTED_WITH' not in self.env:
16781678
logger.error(self._(
16791679
''"csrf X-REQUESTED-WITH xmlrpc required header "
1680-
''"check failed for user%s."),
1681-
current_user)
1680+
''"check failed for user%(userid)s."),
1681+
{"userid": current_user})
16821682
raise UsageError(self._("Required Header Missing"))
16831683

16841684
# Expire old csrf tokens now so we don't use them. These will
@@ -1717,14 +1717,16 @@ def handle_csrf(self, api=False):
17171717
if key is None: # we do not have an @csrf token
17181718
if enforce == 'required':
17191719
logger.error(self._(
1720-
"Required csrf field missing for user%s"), current_user)
1720+
"Required csrf field missing for user%(userid)s"),
1721+
{"userid": current_user})
17211722
raise UsageError(self._(
17221723
''"We can't validate your session (csrf failure). "
17231724
''"Re-enter any unsaved data and try again."))
17241725
if enforce == 'logfailure':
17251726
# FIXME include url
1726-
logger.warning(self._("csrf field not supplied by user%s"),
1727-
current_user)
1727+
logger.warning(self._(
1728+
"csrf field not supplied by user%(userid)s"),
1729+
{"userid": current_user})
17281730
else:
17291731
# enforce is either yes or no. Both permit change if token is
17301732
# missing

website/README.txt

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,17 @@ to free resources:
5757

5858
updating www.roundup-tracker.org
5959
---------------------------------
60+
Note that sourceforge still only has python2 available on it's
61+
systems. Doc updates probably need to be done by building on local
62+
machine using modern Sphinx (7.x or greater) and the generated files
63+
pushed to the website.
64+
65+
Also the files htdocs/ahref* and htdocs/google* have to be copied from
66+
the current production tree to the new directory so we keep some level
67+
of analytics.
68+
69+
-----
70+
6071
Site update requires rebuilding HTML files. For that
6172
`sphinx` is required/
6273
Hopefully, it is already installed into virtualenv, so

website/www/Makefile

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@ help:
1313
# fix that in postprocess
1414
# also sed index.html to properly format meta og:... entries.
1515
html: doc_links ## make standalone HTML files
16-
rm -rf html
16+
if [ -z "${KEEP_HTML}" ]; then rm -rf html; fi
17+
rm -f html/robots.txt # otherwise sphinx errors
1718
mkdir -p $(TMP)/doctrees $(HTML)
1819
sphinx-build -n -W -b html -d $(TMP)/doctrees . $(HTML)
1920
# install searchtools.js into search page.
@@ -23,7 +24,7 @@ html: doc_links ## make standalone HTML files
2324
sed -i -e '/<meta/s/og\\:/og:/' \
2425
-e '/<meta/s/name="og:/property="og:/' html/index.html
2526
cp robots.txt html/robots.txt
26-
mkdir html/signatures && cp signatures/*.asc html/signatures
27+
mkdir -p html/signatures && cp signatures/*.asc html/signatures
2728
cp --no-clobber -r docs/html_extra/. html/docs/.
2829
cp htaccess html/.htaccess
2930
@echo; \

0 commit comments

Comments
 (0)