Page MenuHomePhabricator

D7631.id17223.diff
No OneTemporary

D7631.id17223.diff

Index: src/applications/repository/storage/PhabricatorRepository.php
===================================================================
--- src/applications/repository/storage/PhabricatorRepository.php
+++ src/applications/repository/storage/PhabricatorRepository.php
@@ -580,14 +580,9 @@
// Make sure we don't leak anything if this repo is using HTTP Basic Auth
// with the credentials in the URI or something zany like that.
- if ($uri instanceof PhutilGitURI) {
- if (!$this->getDetail('show-user', false)) {
- $uri->setUser(null);
- }
- } else {
- if (!$this->getDetail('show-user', false)) {
- $uri->setUser(null);
- }
+ // If repository is not accessed over SSH we remove both username and password
+ if (!($uri instanceof PhutilGitURI)) {
+ $uri->setUser(null);
$uri->setPass(null);
}

File Metadata

Mime Type
text/plain
Expires
Wed, Mar 5, 6:15 PM (2 w, 6 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7226204
Default Alt Text
D7631.id17223.diff (870 B)

Event Timeline