Session.fuzz_by_name is deprecated in favor of Session.fuzz(name=name).

[hackathi]

Report

Session.fuzz_by_name unconditionally emits the following warning:

Session.fuzz_by_name is deprecated in favor of Session.fuzz(name=name).

However, if you look at the source code of Session.fuzz(name="something"), you'll find this:

boofuzz/boofuzz/sessions.py

Lines 1280 to 1291 in 5ee41f8

	if name is None or name == "":
	self._main_fuzz_loop(self._generate_mutations_indefinitely(max_depth=max_depth))
	else:
	self.fuzz_by_name(name=name)
	def fuzz_by_name(self, name):
	"""Fuzz a particular test case or node by name.
	Args:
	name (str): Name of node.
	"""
	warnings.warn("Session.fuzz_by_name is deprecated in favor of Session.fuzz(name=name).")

So the "correct" way to call this will trigger the warning.

Expected behavior

No warning if the correct method is used.

Actual behavior

Warning is displayed regardless of method used.

Steps to reproduce the problem

1. use session.fuzz(name="anything")

boofuzz script

No response

boofuzz version

No response

Python version

No response

Platform

No response

Anything else?

Proposed solution: move the code that is in fuzz_by_name() to fuzz(), let fuzz_by_name() call fuzz(name=name) instead.

[jtpereyda]

Thanks for the bug! The proposed solution makes sense to me.