fix: oracle accept list no longer automatically adds blacklisted toke…

…ns (backport #1736) (#1818)

CHANGELOG.md

@@ -48,6 +48,10 @@ Ref: https://keepachangelog.com/en/1.0.0/
 
 ### Fixes
 
+- [1736](https://github.com/umee-network/umee/pull/1736) Blacklisted tokens no longer add themselves back to the oracle accept list.
+
+## [v4.0.1](https://github.com/umee-network/umee/releases/tag/v4.0.1) - 2023-02-10
+
 - [1800](https://github.com/umee-network/umee/pull/1800) Handle non-capitalized assets when calling the historacle data.
 
 ## [v4.0.0](https://github.com/umee-network/umee/releases/tag/v4.0.0) - 2023-01-20

x/oracle/keeper/hooks.go

@@ -24,6 +24,11 @@ func (k Keeper) Hooks() Hooks {
 // it checks if the provided Token should be added to the existing accepted list
 // of assets for the x/oracle module.
 func (h Hooks) AfterTokenRegistered(ctx sdk.Context, token leveragetypes.Token) {
+	if token.Blacklist {
+		// Blacklisted tokens should not trigger updates to the oracle accept list
+		return
+	}
+
 	acceptList := h.k.AcceptList(ctx)
 
 	var tokenExists bool

x/oracle/keeper/hooks_test.go

@@ -24,4 +24,13 @@ func (s *IntegrationTestSuite) TestHooks_AfterTokenRegistered() {
 		Exponent:    6,
 	})
 	s.Require().Len(s.app.OracleKeeper.AcceptList(s.ctx), 3)
+
+	// require a blacklisted token does not update the accept list
+	h.AfterTokenRegistered(s.ctx, leveragetypes.Token{
+		BaseDenom:   "unope",
+		SymbolDenom: "NOPE",
+		Exponent:    6,
+		Blacklist:   true,
+	})
+	s.Require().Len(s.app.OracleKeeper.AcceptList(s.ctx), 3)
 }