Project

General

Profile

Actions

Todo #12431

open

GUI pages should use ``POST`` for AJAX calls, not ``GET``

Added by Jim Pingle over 1 year ago. Updated 4 months ago.

Status:
Pull Request Review
Priority:
Normal
Assignee:
-
Category:
Web Interface
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Plus Target Version:
23.05
Release Notes:
Default

Description

The AJAX buttons on the OpenVPN status page submit values using GET when they should use POST. The variables used in the backend portion of the call use $_REQUEST when they should only use $_POST.

Actions #1

Updated by Viktor Gurov over 1 year ago

also:

pkg.php
services_captiveportal_vouchers.php
vendorstatus_graph.php
vpn_ipsec_phase1.php
status_graph.php
widgets/gmirror_status.widget.php
widgets/interface_statistics.widget.php
widgets/openvpn.widget.php

Actions #2

Updated by Jim Pingle over 1 year ago

  • Subject changed from OpenVPN status page should use ``POST`` for AJAX calls, not ``GET`` to GUI pages should use ``POST`` for AJAX calls, not ``GET``
  • Category changed from OpenVPN to Web Interface

True. May as well fix them all. Updated subject/category.

Actions #4

Updated by Jim Pingle over 1 year ago

  • Status changed from New to Pull Request Review
  • Assignee set to Viktor Gurov
Actions #5

Updated by Jim Pingle about 1 year ago

  • Target version changed from 2.6.0 to CE-Next
  • Plus Target Version changed from 22.01 to 22.05

Need more comprehensive testing.

Actions #6

Updated by Jim Pingle 10 months ago

  • Plus Target Version changed from 22.05 to 22.09
Actions #7

Updated by Jim Pingle 9 months ago

  • Plus Target Version changed from 22.09 to 22.11
Actions #8

Updated by Jim Pingle 5 months ago

  • Plus Target Version changed from 22.11 to 23.01
Actions #9

Updated by Jim Pingle 4 months ago

  • Assignee deleted (Viktor Gurov)
  • Target version changed from CE-Next to 2.7.0
  • Plus Target Version changed from 23.01 to 23.05
Actions

Also available in: Atom PDF