Преглед изворни кода

Merge pull request #3765 from overleaf/jpa-xss-6

[views] mitigate Angular XSS in subscription dashboard

GitOrigin-RevId: 4ac0c431d520957aabd53436ae29a30c2aef3870
Jakob Ackermann пре 5 година
родитељ
комит
58841ecbe9

+ 1 - 1
services/web/app/views/subscriptions/dashboard/_institution_memberships.pug

@@ -8,6 +8,6 @@ each affiliation in confirmedMemberAffiliations
 			|
 			|
 			| #{settings.appName} account as a confirmed member of
 			| #{settings.appName} account as a confirmed member of
 			|
 			|
-			strong= affiliation.institution.name
+			strong(ng-non-bindable)= affiliation.institution.name
 		hr
 		hr
 
 

+ 2 - 2
services/web/app/views/subscriptions/dashboard/_managed_institutions.pug

@@ -5,7 +5,7 @@ each institution in managedInstitutions
 	p
 	p
 		| You are a manager of
 		| You are a manager of
 		|
 		|
-		strong= institution.name
+		strong(ng-non-bindable)= institution.name
 	p
 	p
 		a.btn.btn-primary(href="/metrics/institutions/" + institution.v1Id)
 		a.btn.btn-primary(href="/metrics/institutions/" + institution.v1Id)
 			i.fa.fa-fw.fa-line-chart
 			i.fa.fa-fw.fa-line-chart
@@ -23,7 +23,7 @@ each institution in managedInstitutions
 			| Manage institution managers
 			| Manage institution managers
 	div(ng-controller="MetricsEmailController", ng-cloak)
 	div(ng-controller="MetricsEmailController", ng-cloak)
 		p
 		p
-			span Monthly metrics emails: 
+			span Monthly metrics emails:
 				a(href ng-bind-html="institutionEmailSubscription('"+institution.v1Id+"')" ng-show="!subscriptionChanging" ng-click="changeInstitutionalEmailSubscription('"+institution.v1Id+"')")
 				a(href ng-bind-html="institutionEmailSubscription('"+institution.v1Id+"')" ng-show="!subscriptionChanging" ng-click="changeInstitutionalEmailSubscription('"+institution.v1Id+"')")
 				span(ng-show="subscriptionChanging")
 				span(ng-show="subscriptionChanging")
 					i.fa.fa-spin.fa-refresh(aria-hidden="true")
 					i.fa.fa-spin.fa-refresh(aria-hidden="true")

+ 1 - 1
services/web/app/views/subscriptions/dashboard/_managed_publishers.pug

@@ -2,7 +2,7 @@ each publisher in managedPublishers
 	p
 	p
 		| You are a manager of
 		| You are a manager of
 		|
 		|
-		strong= publisher.name
+		strong(ng-non-bindable)= publisher.name
 	p
 	p
 		a(href="/publishers/" + publisher.slug + "/hub")
 		a(href="/publishers/" + publisher.slug + "/hub")
 			i.fa.fa-fw.fa-user-circle
 			i.fa.fa-fw.fa-user-circle

+ 2 - 2
services/web/app/views/subscriptions/dashboard/_team_name_mixin.pug

@@ -1,9 +1,9 @@
 mixin teamName(subscription)
 mixin teamName(subscription)
 	- if (subscription.teamName && subscription.teamName != '')
 	- if (subscription.teamName && subscription.teamName != '')
-		strong= subscription.teamName
+		strong(ng-non-bindable)= subscription.teamName
 	- else if (subscription.admin_id._id == user._id)
 	- else if (subscription.admin_id._id == user._id)
 		| a group account
 		| a group account
 	- else
 	- else
 		| the group account owned by
 		| the group account owned by
 		|
 		|
-		strong= subscription.admin_id.email
+		strong= subscription.admin_id.email