Skip to content

Commit

Permalink
okhttp: ignore unknown HTTP/2 settings
Browse files Browse the repository at this point in the history
  • Loading branch information
ericgribkoff authored May 30, 2017
1 parent ee38b37 commit f52b4c1
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -301,7 +301,8 @@ private void readSettings(Handler handler, int length, byte flags, int streamId)
case 6: // SETTINGS_MAX_HEADER_LIST_SIZE
break; // Advisory only, so ignored.
default:
throw ioException("PROTOCOL_ERROR invalid settings id: %s", id);
// Implementations MUST ignore any unknown or unsupported identifier.
continue;
}
settings.set(id, 0, value);
}
Expand Down

0 comments on commit f52b4c1

Please sign in to comment.