Project

General

Profile

Actions

Bug #21638

closed

IP-lock not aware of IPv6

Added by Bas v.d. Wiel over 13 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Locking / Session Handling
Target version:
-
Start date:
2010-08-06
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
4.7
PHP Version:
5.3
Tags:
Complexity:
medium
Is Regression:
No
Sprint Focus:

Description

When locking a session to an IP with less than 4 parts, the core attempts to do IPv4 things to a potentially IPv6 address.

The attached patch adds a simple check for IPv6 addresses and keeps TYPO3 from wrongly assuming IPv4. It just returns the full IP, assuming this choice will lock the session to the individual address (reproducing the default strictest behaviour for IPv4).

The patch is against class.t3lib_userauth.php, being the first actual patch I ever contributed to another project.. please go easy on me if I did it wrong. ;-)
(issue imported from #M15379)


Files

userauth_ipv6.patch (239 Bytes) userauth_ipv6.patch Administrator Admin, 2010-08-06 18:49
15379_v2.diff (565 Bytes) 15379_v2.diff Administrator Admin, 2010-08-08 12:50

Related issues 5 (0 open5 closed)

Related to TYPO3 Core - Feature #25336: Add support for IPv6Closed2011-03-16

Actions
Related to TYPO3 Core - Bug #89226: Removed IPlock properties are still assignedClosedBenni Mack2019-09-23

Actions
Has duplicate TYPO3 Core - Bug #58699: IPv6 completely ignored by FE/BE lockIPClosed2014-05-11

Actions
Has duplicate TYPO3 Core - Feature #62375: IPv6 session lock Closed2014-10-21

Actions
Has duplicate TYPO3 Core - Bug #44109: ses_iplock IPv6Closed2012-12-19

Actions
Actions #1

Updated by Steffen Gebert over 13 years ago

Hi Bas,

thanks for your patch!
I created a unified diff of your changes, which is what we need for the review process.

Could you please check this and send it to the core-team mailinglist afterwards, to integrate it into the next release of TYPO3?

Please read http://typo3.org/teams/core/core-mailinglist-rules/ and use TYPO3 4-4 and trunk as affected branches.

Thanks
Steffen

Actions #2

Updated by Bas v.d. Wiel over 13 years ago

Hi Steffen,

Thanks for correcting my patch. I can see how it was done, so I'll stick to this procedure next time. Reading up on core list rules now..

Best regards,
Bas

Actions #3

Updated by Susanne Moog over 12 years ago

  • Status changed from Needs Feedback to Accepted
  • Target version deleted (0)
  • TYPO3 Version changed from 4.5 to 4.7
  • Complexity set to medium
Actions #4

Updated by Markus Klein almost 10 years ago

  • Priority changed from Should have to Must have
  • Is Regression set to No
Actions #5

Updated by Mathias Schreiber about 9 years ago

  • Target version set to 7.4 (Backend)
Actions #6

Updated by Susanne Moog over 8 years ago

  • Target version changed from 7.4 (Backend) to 7.5
Actions #7

Updated by Benni Mack over 8 years ago

  • Target version deleted (7.5)
Actions #8

Updated by Gerrit Code Review almost 7 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52947

Actions #9

Updated by Susanne Moog over 6 years ago

  • Category set to Locking / Session Handling
Actions #10

Updated by Gerrit Code Review over 6 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52947

Actions #11

Updated by Gerrit Code Review over 6 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52947

Actions #12

Updated by Gerrit Code Review over 6 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52947

Actions #13

Updated by Gerrit Code Review about 6 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52947

Actions #14

Updated by Gerrit Code Review about 6 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52947

Actions #15

Updated by Gerrit Code Review about 6 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52947

Actions #16

Updated by Gerrit Code Review over 5 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52947

Actions #17

Updated by Gerrit Code Review over 5 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52947

Actions #18

Updated by Gerrit Code Review over 5 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52947

Actions #19

Updated by Gerrit Code Review over 5 years ago

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52947

Actions #20

Updated by Gerrit Code Review over 5 years ago

Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52947

Actions #21

Updated by Gerrit Code Review over 5 years ago

Patch set 13 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52947

Actions #22

Updated by Gerrit Code Review almost 5 years ago

Patch set 14 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #23

Updated by Gerrit Code Review almost 5 years ago

Patch set 15 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #24

Updated by Gerrit Code Review over 4 years ago

Patch set 16 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #25

Updated by Gerrit Code Review over 4 years ago

Patch set 17 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #26

Updated by Gerrit Code Review over 4 years ago

Patch set 18 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #27

Updated by Gerrit Code Review over 4 years ago

Patch set 19 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #28

Updated by Gerrit Code Review over 4 years ago

Patch set 20 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #29

Updated by Gerrit Code Review over 4 years ago

Patch set 21 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #30

Updated by Gerrit Code Review over 4 years ago

Patch set 22 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #31

Updated by Gerrit Code Review over 4 years ago

Patch set 23 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #32

Updated by Gerrit Code Review over 4 years ago

Patch set 24 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #33

Updated by Gerrit Code Review over 4 years ago

Patch set 25 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #34

Updated by Gerrit Code Review over 4 years ago

Patch set 26 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #35

Updated by Gerrit Code Review over 4 years ago

Patch set 27 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #36

Updated by Gerrit Code Review over 4 years ago

Patch set 28 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/52947

Actions #37

Updated by Alexander Stehlik over 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #38

Updated by Benni Mack over 4 years ago

  • Status changed from Resolved to Closed
Actions #39

Updated by Benni Mack over 4 years ago

  • Related to Bug #89226: Removed IPlock properties are still assigned added
Actions

Also available in: Atom PDF